config: make numeric parsing errors more clear
commit2f666581bbc1895cf66a7007fb222026b299deb9
authorJeff King <peff@peff.net>
Sun, 8 Sep 2013 08:38:22 +0000 (8 04:38 -0400)
committerJunio C Hamano <gitster@pobox.com>
Mon, 9 Sep 2013 18:07:07 +0000 (9 11:07 -0700)
tree7a5dd7bdbfc6f99a060ec263330a21184c26a34a
parent33fdd77e2b8ece3490982f9a35c8669d16879ba8
config: make numeric parsing errors more clear

If we try to parse an integer config argument and get a
number outside of the representable range, we die with the
cryptic message: "bad config value for '%s'".

We can improve two things:

  1. Show the value that produced the error (e.g., bad
     config value '3g' for 'foo.bar').

  2. Mention the reason the value was rejected (e.g.,
     "invalid unit" versus "out of range").

A few tests need to be updated with the new output, but that
should not be representative of real-world breakage, as
scripts should not be depending on the exact text of our
stderr output, which is subject to i18n anyway.

Signed-off-by: Jeff King <peff@peff.net>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
config.c
t/t1300-repo-config.sh
t/t4055-diff-context.sh