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

Re: [PATCH] Re: replacing SVN_REVNUM_T_FMT for translation

From: Tobias Ringström <tobias_at_ringstrom.mine.nu>
Date: 2004-05-13 13:32:01 CEST

Branko Čibej wrote:
> I'll note that most of our sprintfs are 100% safe. I see no reason to
> eliminate those, and add more allocations or gods forbid new pool
> arguments where there weren't any.

I checked the sprintfs that Peter changed and they were not in a
performance critical path, but I agree that there is no real need to
change them either. I have no strong opinion either way in this case,
but I did review his patch and it was at least correct.


To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Thu May 13 13:33:30 2004

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.