Re: [PATCH 2/2] Move unsigned long option parsing out of pack-objects.c

2015-06-22 Thread Junio C Hamano
Charles Bailey char...@hashpling.org writes: On Sun, Jun 21, 2015 at 07:25:44PM +0100, Charles Bailey wrote: From: Charles Bailey cbaile...@bloomberg.net diff --git a/parse-options.c b/parse-options.c index 80106c0..101b649 100644 --- a/parse-options.c +++ b/parse-options.c @@ -180,6

Re: [PATCH 2/2] Move unsigned long option parsing out of pack-objects.c

2015-06-22 Thread Junio C Hamano
Charles Bailey char...@hashpling.org writes: From: Charles Bailey cbaile...@bloomberg.net The unsigned long option parsing (including 'k'/'m'/'g' suffix parsing) is more widely applicable. Add support for OPT_MAGNITUDE to parse-options.h and change pack-objects.c use this support. The

Re: [PATCH 2/2] Move unsigned long option parsing out of pack-objects.c

2015-06-21 Thread Charles Bailey
On Sun, Jun 21, 2015 at 07:25:44PM +0100, Charles Bailey wrote: From: Charles Bailey cbaile...@bloomberg.net diff --git a/parse-options.c b/parse-options.c index 80106c0..101b649 100644 --- a/parse-options.c +++ b/parse-options.c @@ -180,6 +180,23 @@ static int get_value(struct

[PATCH 2/2] Move unsigned long option parsing out of pack-objects.c

2015-06-21 Thread Charles Bailey
From: Charles Bailey cbaile...@bloomberg.net The unsigned long option parsing (including 'k'/'m'/'g' suffix parsing) is more widely applicable. Add support for OPT_MAGNITUDE to parse-options.h and change pack-objects.c use this support. The error behavior on parse errors follows that of