On Thu, May 26, 2016 at 2:02 AM, Remi Galan Alfonso
<remi.galan-alfonso@xxxxxxxxxxxxxxxxxxxxxxx> wrote:
> Hi Stefan,
>
> Stefan Beller <sbeller@xxxxxxxxxx> writes:
>> [...]
>> @ -353,6 +354,15 @@ static int parse_config(const char *var, const char *value, void *data)
>> else if (parse_submodule_update_strategy(value,
>> &submodule->update_strategy) < 0)
>> die(_("invalid value for %s"), var);
>> + } else if (!strcmp(item.buf, "shallow")) {
>> + if (!me->overwrite &&
>> + submodule->recommend_shallow != -1)
>
> Nit: You seems to be able to keep the whole condition on the same line:
>
> if (!me->overwrite && submodule->recommend_shallow != -1)
>
> If you want to keep it in two line, you might want to align it:
> if (!me->overwrite &&
> submodule->recommend_shallow != -1)
Thanks will fix!
>
> Thanks,
> Rémi
--
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