[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

[PATCH] Simplify and optimize RA-svn's string marshalling

From: Julian Foad <julian.foad_at_wandisco.com>
Date: Fri, 03 Jun 2011 09:52:21 +0100

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.

- Julian

Received on 2011-06-03 10:53:00 CEST

This is an archived mail posted to the Subversion Dev mailing list.

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.