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