Re: [nftables] TODO: Replace yy_switch_to_buffer by yypop_buffer_state and yypush_buffer_state

[Florian Westphal <fw@strlen.de>]

Gopal Yadav <gopunop@gmail.com> wrote:
> Hi Netfilter Team,
> 
> I am looking to resolve a todo task in function yy_switch_to_buffer in
> nftables/src/scanner.c file.
> I am not familiar with lex but searching around I found that scanner.c
> is produced by lex according to the scanner.l file.
> Therefore my guess is, changing the scanner.c file directly is not the
> solution, right?

Right.

> Changes would have to be done in scanner.l, right?

No.

> How should I proceed to complete this todo?

This TODO is coming from flex itself, so, this is not an nftables
task.

> I browsed bugzilla to find some other issues to solve, but I feel
> lost. Are there any beginner friendly issues to solve or any other
> starting point?

This one for example:
https://bugzilla.netfilter.org/show_bug.cgi?id=1305

Its "just" a documentation issue.  You could work from comment 4
and improve the nft documentation to clarify 'accept' behaviour.

For many other bugs it would help if we had testcases that demonstrated
this problem in the nftables.git repo.

So, if you can translate a BZ ticker to e.g. a new test case in
tests/shell that show problem still exists in current nftables.git then
you could submit that test case as a patch, even if the problem is not
yet resolved.

Readily available test reproducers help a lot.