Hello,
sorry it dropped on the floor of my INBOX...
thanks for reminding.
> Yes, the main anchor prints as "" but all that is behind compile time
> -DOPT_DEBUG so regular users won't deal with it anyway, so keep the code
> simple instead of adding logging around `rs->anchor->path'.
>
> OK?
cha
Klemens Nanni writes:
> On Wed, Nov 27, 2019 at 08:04:47PM +0100, Klemens Nanni wrote:
>> If an anchor/ruleset contains no rules, there is no point in creating
>> a temporary copy, optimizing and replacing it.
>>
>> Regress passes on amd64.
>>
>> Feedback? OK?
> Anyone?
>
FWIW, it looks good
On Wed, Nov 27, 2019 at 08:04:47PM +0100, Klemens Nanni wrote:
> If an anchor/ruleset contains no rules, there is no point in creating
> a temporary copy, optimizing and replacing it.
>
> Regress passes on amd64.
>
> Feedback? OK?
Anyone?
All optimizations work on actual rules; if there are non
If an anchor/ruleset contains no rules, there is no point in creating
a temporary copy, optimizing and replacing it.
Regress passes on amd64.
Feedback? OK?
Index: pfctl_optimize.c
===
RCS file: /cvs/src/sbin/pfctl/pfctl_optimize.c,