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

Re: svn commit: r1189103 - /subversion/trunk/subversion/libsvn_subr/mergeinfo.c

From: Philip Martin <philip.martin_at_wandisco.com>
Date: Wed, 26 Oct 2011 13:58:34 +0100

Hyrum K Wright <hyrum.wright_at_wandisco.com> writes:

> While I appreciate the need to have concise code, it wouldn't surprise me if
> the special case was in there for performance reasons. If the no-mergeinfo
> case is very common, avoiding superfluous calls to mergeinfo_to_string()
> and svn_stringbuf__morph_into_string() could have quite an impact.

mergeinfo_to_stringbuf has the same optimisation.

-- 
uberSVN: Apache Subversion Made Easy
http://www.uberSVN.com
Received on 2011-10-26 14:59:10 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.