From mboxrd@z Thu Jan 1 00:00:00 1970 From: Patrick McHardy Subject: Re: [PATCH libnftables] Add support for ct set Date: Fri, 10 Jan 2014 13:58:01 +0000 Message-ID: <20140110135800.GA8854@macbook.localnet> References: <1389359425-6837-1-git-send-email-kristian.evensen@gmail.com> <20140110131406.GA8088@macbook.localnet> <20140110132703.GA8224@macbook.localnet> <20140110134342.GA8720@macbook.localnet> <20140110135435.GA18191@localhost> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: Kristian Evensen , Netfilter Development Mailing list To: Pablo Neira Ayuso Return-path: Received: from stinky.trash.net ([213.144.137.162]:59385 "EHLO stinky.trash.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753379AbaAJN6F (ORCPT ); Fri, 10 Jan 2014 08:58:05 -0500 Content-Disposition: inline In-Reply-To: <20140110135435.GA18191@localhost> Sender: netfilter-devel-owner@vger.kernel.org List-ID: On Fri, Jan 10, 2014 at 02:54:35PM +0100, Pablo Neira Ayuso wrote: > On Fri, Jan 10, 2014 at 01:43:43PM +0000, Patrick McHardy wrote: > > On Fri, Jan 10, 2014 at 02:33:06PM +0100, Kristian Evensen wrote: > > > Hi, > > > > > > On Fri, Jan 10, 2014 at 2:27 PM, Patrick McHardy wrote: > > > > No, I'm refering to the (ab)use of the expression. Anything not returning > > > > data is not an expression but a statement. > > > > > > Ok, then I follow :) I followed the naming in meta, but I agree. What > > > would be a good naming convetion? I thought of something like > > > nft_expr_stmt_*. It is a bit clumsy, but it is at least clear that the > > > struct can be used to represent both an expression and a statement. > > > > nft_ct_stmt? This is what we use in nftables f.i. in case of meta. > > Perhaps nft_ct_instr? So we can identify this as the nftables > instruction-set. Well, expressions also belong to the instruction set. A statement is is one (more specific) case of an instruction, as are expressions. Why introduce new terminology that isn't used anywhere else so far if statement is the exact description of what this is and is already used by nftables.