Re: [PATCH] Add support for the 'unless' matcher in the dynamic layer.

2016-07-26 Thread Zac Hansen via cfe-commits
Even if it still did add overhead, it seems perfectly reasonable, from a user's perspective (namely mine), that if I introduce unnecessary narrowing matchers to my chain that there may be a performance penalty. The ability to do the following easily outweighs any performance issues for me: anyOf

Re: [PATCH] Add support for the 'unless' matcher in the dynamic layer.

2016-07-26 Thread Samuel Benzaquen via cfe-commits
One of the reasons we added the minimum was because these nodes added overhead to the matching that was not unnecessary when they only had a single node. On the current implementation we could actually get rid of the node completely for the one argument calls. I would be ok with removing the lower