[PATCH] Use Quicksearch in strstr

[Zack Weinberg]

On Mon, Oct 29, 2018 at 6:54 PM Ondřej Bílka <neleai@seznam.cz> wrote:
> Real benefit is fixing idiocy
...
> That "benchmark" is complete garbage
...

When you dismiss other people's testing this aggressively you make
them not want to cooperate with you.  If I had written this patch, my
reaction to this feedback would be: Fine, fix it yourself if you think
my approach is idiotic.

It is perfectly possible to express concerns in a less hostile manner.
For instance, you could have said "I think this benchmark doesn't
account for the cost of branch misprediction, because it calls strstr
many times with the same inputs.  Can you please test it on a wider
variety of inputs in a randomized order?"

zw