Re: [PATCH nft] src: replace struct stmt_ops by type field in struct stmt

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Pablo Neira Ayuso <pablo@xxxxxxxxxxxxx> wrote:
> Shrink struct stmt in 8 bytes.
> 
> __stmt_ops_by_type() provides an operation for STMT_INVALID since this
> is required by -o/--optimize.

We could have lots of statements in a ruleset, so I think this makes
sense to compact things.
> 
> Similar to:
> 
>  68e76238749f ("src: expr: add and use expr_name helper").
>  72931553828a ("src: expr: add expression etype")
>  2cc91e6198e7 ("src: expr: add and use internal expr_ops helper")

... and it makes statements more like expressions wrt. ops, so this is
good too.

Acked-by: Florian Westphal <fw@xxxxxxxxx>





[Index of Archives]     [Netfitler Users]     [Berkeley Packet Filter]     [LARTC]     [Bugtraq]     [Yosemite Forum]

  Powered by Linux