On Wed, Mar 18, 2015 at 4:47 PM, Michael Haggerty [off-list ref] wrote:
quoted
Thank you for doing it. I was about to write another number parser and
you did it :D Maybe you can add another patch to convert the only
strtol in upload-pack.c to parse_ui. This place should accept positive
number in base 10, plus sign is not accepted.
If the general direction of this patch series is accepted, I'll
gradually try to go through the codebase, replacing *all* integer
parsing with these functions. So there's no need to request particular
callers of strtol()/strtoul() to be converted; I'll get to them all
sooner or later (I hope).
Good to know. No there's no hurry in converting this particular place.
It's just tightening things up, not bug fixing or anything.
--
Duy