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

Re: svn commit: r23307 - trunk/subversion/svn

From: Hyrum K. Wright <hyrum_wright_at_mail.utexas.edu>
Date: 2007-02-01 20:57:46 CET

Peter Lundblad wrote:
> Hyrum K. Wright writes:
> ...
>>> SVN_REVNUM_T_FMT was kept as a convenience for external library users.
>> I agree that we shouldn't be using SVN_REVNUM_T_FMT in places that break gettext.
>> The line that Dan pointed out just converts the revision number to a string,
>> which is then used elsewhere. There are several places we do this, and I've
>> already made the change in r23309. If we still think that we shouldn't use
>> SVN_REVNUM_T_FMT in any of these locations, I'll be happy to revert.
>>
>
> Yeah, please revert that change again! There's absolutely no point in
> using a constant here since the compiler (GCC at least) already checks
> the correctness of the format string.
>
> Sorry for the extra work,

No problem. Thanks for the review.

I've reverted the change in r23323.

-Hyrum

Received on Thu Feb 1 20:58:14 2007

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.