Re: [PATCH] Improve single byte read stream performance

Johan Corveleyn <jcorvel_at_gmail.com> writes:

> On Tue, Mar 9, 2010 at 8:04 PM, Julian Foad <julian.foad_at_wandisco.com> wrote:
>> On Sun, 2010-03-07, Stefan Fuhrman wrote:
>>> Speed up input stream processing in config parser and
>>> others that read single bytes from a stream.
>>>
>>> * subversion/libsvn_subr/subst.c
>>>   (translated_stream_read): Add an optimized code path
>>>     for single byte read requests.
>>>
>>> Patch by: Stefan Fuhrmann <stefanfuhrmann{_AT_}alice-dsl.de>
>>
>> Committed in r921057.
>
> Any chance this could be backported to 1.6.x?
>
> This seems like a nice performance improvement with low impact on the
> rest of the code. Would be nice if this could be put out there as
> quickly as possible (i.e. not having to wait for 1.7) for users to
> enjoy ...

Can you quantify the improvement? I don't suppose a GUI like
TortoiseSVN reads config files very often. Any improvement is lost in
the noise when I run the command line client on Linux. It doesn't
change the number of system calls reported by 'strace -c'.

-- 
Philip