On 7/21/2014 7:15 PM, Matthieu Moy wrote:
> Tanay Abhra <tanay...@gmail.com> writes:
>> On 7/21/2014 6:21 PM, Matthieu Moy wrote:
>>> 2) Add a by-address parameter to git_configset_get_value that allows the
>>>    user to get the file and line information. In your previous patch,
>>>    that would mean returning a pointer to the corresponding struct
>>>    key_source.
>> Will this extra complexity be good for "git_configset_get_value"?
>> Instead can we provide a function like die_config(char *key)
>> which prints
>>      die("bad config file line %d in %s", linenr, filename);?
> Where would you call this function, and where would you take linenr and
> filename?

Usage can be like this,

if(!git_config_get_value(k, &v)) {
        if (!v) {
                /* die_config calls git_config_get_value_multi for 'k',
                 * gets the string list with the util pointer containing
                 * the linenr and the file name, dies printing the message.
        } else
                /* do work */

Above example works just like the current code. Currently the callbacks
does not have the access to the linenr and file name anyway.
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