Tanay Abhra <tanay...@gmail.com> writes:

> +void git_die_config(const char *key)
> +{
> +     const struct string_list *values;
> +     struct key_value_info *kv_info;
> +     values = git_config_get_value_multi(key);
> +     kv_info = values->items[values->nr - 1].util;
> +     if (!kv_info->linenr)
> +             die(_("unable to parse '%s' from command-line config"), key);
> +     else
> +             die(_("bad config variable '%s' at file line %d in %s"),
> +                     key,
> +                     kv_info->linenr,
> +                     kv_info->filename);
> + }

Extra whitespace before }.

Also, didn't we agree that it was a good thing to factor this
if/then/else into a helper function?

-- 
Matthieu Moy
http://www-verimag.imag.fr/~moy/
--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to