Re: [PATCH] sprintf elimination, take 2

> > statements inside the if(detailed) clause). I wonder why you didn't
> > change all sprintf's throughout the code, e.g. those in
> libsvn_repos/dump.c?
> >
> Did in that file too, now. NOw we just have sprintf in test cases (who
> cares?), svndumpfilter/main.c (missing pool arguments) and utf.c and
> path.c. The two last construct strings in place and carefully make sure
> that enough space is allocated. I think extra allocation there is
> overkill.

Changing the function signature in svndumpfilter/main.c would be fine.
There is no change in the public API by adding a pool paramater to a private
function. I should have been more exact.

bye,

Erik.

-- 
NEU : GMX Internet.FreeDSL
Ab sofort DSL-Tarif ohne Grundgebühr: http://www.gmx.net/dsl
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org