Tanay Abhra <tanayabh@xxxxxxxxx> writes: > On 7/24/2014 9:09 PM, Matthieu Moy wrote: >> Tanay Abhra <tanayabh@xxxxxxxxx> writes: >> >>> I am attaching the v12 with two new functions git_configset_get_string() & >>> git_configset_get_string_const(). >> >> Didn't you intend to change git_config_get_string to let it die in case >> of error instead of returning an error code? >> > > Yup, but it is the part of the new series not the old one. > I sent this revised patch just for solving the constness > debate. > Also, I would need git_die_config() to print a proper > error message which comes in the 7 part series not this one. Ah, right, you don't have line numbers yet, so you can't die properly. So, hopefully, this part can be considered done, and we can continue building on top. As a side effect, I guess Junio will apply this on top of master so the string interning API will be available immediately at the tip of the branch. -- 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