Jeff King <peff@xxxxxxxx> writes:
> -- >8 --
> Subject: drop odd return value semantics from userdiff_config
> ...
> 5 files changed, 12 insertions(+), 37 deletions(-)
>
> diff --git a/builtin/blame.c b/builtin/blame.c
> index 5a67c20..01956c8 100644
> --- a/builtin/blame.c
> +++ b/builtin/blame.c
> @@ -2050,14 +2050,8 @@ static int git_blame_config(const char *var, const char *value, void *cb)
> return 0;
> }
>
> - switch (userdiff_config(var, value)) {
> - case 0:
> - break;
> - case -1:
> + if (userdiff_config(var, value) < 0)
> return -1;
> - default:
> - return 0;
> - }
>
> return git_default_config(var, value, cb);
> }
> ...
Looks very nice ;-)
--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html