Tanay Abhra <tanayabh@xxxxxxxxx> writes: > [PATCH v4]: Tiny style nits corrected. Patch 2/5 has been totally reworked. > One thing to check is if the config variables I changed in the series > are single valued or multi valued. Though I have tried to ascertain > if the variable was single valued or not by reading the docs and code, > mistakes may creep in. > > [PATCH v3]: Most of Eric's suggestions has been implemented. See [2] for discussion. > Also, new helpers introduced in v7 of the config-set API series have been used. > > This series builds on the top of 4c715ebb in pu (ta/config-set). I think it semantically needs your other WIP series, that makes git_config_get_* die in case of error. Otherwise, there's an unexpected behavior change in case of error. -- 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 majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html