Re: [nft PATCH] Enable automerge feature for anonymous sets

2018-02-15 Thread Pablo Neira Ayuso
On Tue, Feb 06, 2018 at 07:18:47PM +0100, Phil Sutter wrote: > Automatic merging of adjacent/overlapping ranges upon insertion has > clear benefits performance- and readability-wise. The drawbacks which > led to disabling it by default don't apply to anonymous sets since they > are read-only anyway

Re: [nft PATCH] Enable automerge feature for anonymous sets

2018-02-07 Thread Phil Sutter
Hi Pablo, On Wed, Feb 07, 2018 at 12:39:43AM +0100, Pablo Neira Ayuso wrote: > On Tue, Feb 06, 2018 at 07:18:47PM +0100, Phil Sutter wrote: > > Automatic merging of adjacent/overlapping ranges upon insertion has > > clear benefits performance- and readability-wise. The drawbacks which > > led to d

Re: [nft PATCH] Enable automerge feature for anonymous sets

2018-02-07 Thread Jozsef Kadlecsik
On Wed, 7 Feb 2018, Pablo Neira Ayuso wrote: > On Tue, Feb 06, 2018 at 07:18:47PM +0100, Phil Sutter wrote: > > Automatic merging of adjacent/overlapping ranges upon insertion has > > clear benefits performance- and readability-wise. The drawbacks which > > led to disabling it by default don't app

Re: [nft PATCH] Enable automerge feature for anonymous sets

2018-02-06 Thread Pablo Neira Ayuso
Hi Phil, On Tue, Feb 06, 2018 at 07:18:47PM +0100, Phil Sutter wrote: > Automatic merging of adjacent/overlapping ranges upon insertion has > clear benefits performance- and readability-wise. The drawbacks which > led to disabling it by default don't apply to anonymous sets since they > are read-o

[nft PATCH] Enable automerge feature for anonymous sets

2018-02-06 Thread Phil Sutter
Automatic merging of adjacent/overlapping ranges upon insertion has clear benefits performance- and readability-wise. The drawbacks which led to disabling it by default don't apply to anonymous sets since they are read-only anyway, so enable this feature for them again. Cc: Jeff Kletsky Signed-of