Re: [PATCH v5 3/7] change `git_config()` return value to void

2014-07-31 Thread Tanay Abhra
On 7/31/2014 4:52 PM, Matthieu Moy wrote: > Tanay Abhra writes: > >> Signed-off-by: Tanay Abhra > > I think I deserve a bit of credit here ;-). > Yes, but to show credit would I have to write "from you" or "signed-off-by"? :) >> { >> -return git_config_with_options(fn, data, NULL, 1);

Re: [PATCH v5 3/7] change `git_config()` return value to void

2014-07-31 Thread Matthieu Moy
Tanay Abhra writes: > On 7/31/2014 4:52 PM, Matthieu Moy wrote: >> Tanay Abhra writes: >> >>> Signed-off-by: Tanay Abhra >> >> I think I deserve a bit of credit here ;-). >> > > Yes, but to show credit would I have to write "from you" or > "signed-off-by"? :) Original-patch-by: would be fine

Re: [PATCH v5 3/7] change `git_config()` return value to void

2014-07-31 Thread Matthieu Moy
Matthieu Moy writes: > Tanay Abhra writes: > >> Signed-off-by: Tanay Abhra > > I think I deserve a bit of credit here ;-). > >> { >> -return git_config_with_options(fn, data, NULL, 1); >> +if (git_config_with_options(fn, data, NULL, 1) < 0) >> +/* >> + * git_con

Re: [PATCH v5 3/7] change `git_config()` return value to void

2014-07-31 Thread Matthieu Moy
Tanay Abhra writes: > Signed-off-by: Tanay Abhra I think I deserve a bit of credit here ;-). > { > - return git_config_with_options(fn, data, NULL, 1); > + if (git_config_with_options(fn, data, NULL, 1) < 0) > + /* > + * git_config_with_options() normally retu

[PATCH v5 3/7] change `git_config()` return value to void

2014-07-31 Thread Tanay Abhra
Currently `git_config()` returns an integer signifying an error code. During a previous rewrite of the function most of the code was shifted to `git_config_with_options()`. `git_config_with_options()` normally returns positive values if its `config_source` parameter is set as NULL, as most errors a