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

Re: [PATCH] issue 1780: Keyword values with dollar signs cause badness.

From: John Peacock <jpeacock_at_rowman.com>
Date: 2006-01-10 03:22:42 CET

Lieven Govaerts wrote:
> Can you agree with this change in algorithm? I don't see any problems in
> the algorithm itself, but there may be some work on the implementation.
> Performance impact is very low, due to the max. keyword length of 255 chars.

At a cursory glance, it seems like the change would work, but one thing you may
not be aware of was that SVN_KEYWORD_MAX_LEN #define may change. The reason is
that it is surprisingly easy to get a path longer than 255 characters in big
projects (like GCC, to use a real life example). I don't know if the #define is
going to be eliminated entirely or just increase by a couple powers of two.
Some testing is going to have to take place as to whether this extra looping
will have any performance issues.

In any case, some test cases and a good log message (see HACKING) are essential
to get a thorough review...


John Peacock
Director of Information Research and Technology
Rowman & Littlefield Publishing Group
4720 Boston Way
Lanham, MD 20706
301-459-3366 x.5010
fax 301-429-5747
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Tue Jan 10 03:27:54 2006

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.