[PATCH] speed up tokenize_command
Marc at facebook.com
Mon Apr 16 22:59:09 UTC 2007
OK I took a closer look. Looks like it¹s faster for O3 and it¹s cleaner
too! I¹ll check with Steve about rolling it in. Thanks.
On 4/12/07 4:57 PM, "Paolo Borelli" <paolo.borelli at gmail.com> wrote:
> Hi, thanks for integrating the previous patches so fast!
> It made me want to stare a bit at the code to look for any other little
> improvements to make. I stumbled on tokenize_command() and for fun I
> tried to write a faster version (no idea if it is a real bottleneck).
> Attached there is my version and the small bench I used: on my machine
> the function is about 20% faster. The nice thing is that the function is
> even shorter and cleaner :)
> I tested it a bit to see if my function handled all the cases in the
> same way as the old one and it seems ok, but keep in mind that it has
> not been tested in production...
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the memcached