Re: [PATCH 14/14] diff_opt_parse(): use convert_i() when handling --abbrev=

2015-03-18 Thread Junio C Hamano
Michael Haggerty writes: > die() with an error message if the argument is not a non-negative > integer. This change tightens up parsing: '+' and '-', leading > whitespace, and trailing junk are all disallowed now. > > Signed-off-by: Michael Haggerty > --- > diff.c | 3 ++- > 1 file changed, 2 i

[PATCH 14/14] diff_opt_parse(): use convert_i() when handling --abbrev=

2015-03-17 Thread Michael Haggerty
die() with an error message if the argument is not a non-negative integer. This change tightens up parsing: '+' and '-', leading whitespace, and trailing junk are all disallowed now. Signed-off-by: Michael Haggerty --- diff.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/