[PATCH v6 4/9] Improve fd filtering

[Nikolay Marchuk]

On 07.08.2017 11:52, Eugene Syromiatnikov wrote:
> What makes incredibly difficult to review this patch is the abundance of
> all these syscall sets.  I think, making return code of
> match_fd_common/pathtrace_match_state tri-state (positive match,
> negative match, no match) instead of boolean could significantly
> simplify the code and avoid the duplication in run_fd_filter and
> pathtrace_match_set.

I had already tried to made tri-state implementation of match_fd_common,
but in my implementation match_fd_common had additional
responsibilities from pathtrace_match_set and code didn't become simpler.
I have no idea what states should I choose to simplify code.

> ------------------------------------------------------------------------------
> Check out the vibrant tech community on one of the world's most
> engaging tech sites, Slashdot.org! http://sdm.link/slashdot
> _______________________________________________
> Strace-devel mailing list
> Strace-devel at lists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/strace-devel
>

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 228 bytes
Desc: OpenPGP digital signature
URL: <http://lists.strace.io/pipermail/strace-devel/attachments/20170807/072cd290/attachment.bin>