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.
