On 03.06.2011 10:52, Julian Foad wrote:
> Hi Stefan2 and others.
>
> This patch simplifies code in RA-svn/marshal.c by using a single code
> path to read both short and long strings efficiently. Using a single
> code path is beneficial for test coverage.
>
> It is an alternative to r1028352 which merged r985606 and r1028092 from
> the performance branch.
>
> I have run the test suite over RA-svn (with BDB) but haven't done any
> more testing than that. It's just something I noticed while looking at
> that code, it's not solving a problem I've noticed or anything like
> that. By inspection I believe it is more efficient than
> read_long_string() was, and unchanged for short strings, but if you're
> able to do any performance analysis on it, that would be great.
Following the discussion elsewhere in this thread,
I decided not to limit the size of strings transmittable
over svn just yet. Let's focus on getting 1.7 out of
the door without introducing new pitfalls ;)
Committed as r1135210 (after some fine-tuning).
-- Stefan^2.
Received on 2011-06-14 20:16:20 CEST