Message ID | 1369835434-27727-2-git-send-email-thomas@codesourcery.com |
---|---|
State | New |
Headers | show |
On 29 May 2013 14:50, Thomas Schwinge <thomas@codesourcery.com> wrote: > Signed-off-by: Thomas Schwinge <thomas@codesourcery.com> Reviewed-by: Peter Maydell <peter.maydell@linaro.org> > --- > util/envlist.c | 5 +++-- > 1 file changed, 3 insertions(+), 2 deletions(-) > > diff --git util/envlist.c util/envlist.c > index ebc06cf..cbbf7e5 100644 > --- util/envlist.c > +++ util/envlist.c NB: your process for generating patch mails seems to be slightly wrong -- the diff header in a patch mail should look like this: diff --git a/util/envlist.c b/util/envlist.c index ebc06cf..cbbf7e5 100644 --- a/util/envlist.c +++ b/util/envlist.c (note the extra 'a' and 'b'); otherwise 'git am patch.mbox' will complain when you try to apply the patch. thanks -- PMM
diff --git util/envlist.c util/envlist.c index ebc06cf..cbbf7e5 100644 --- util/envlist.c +++ util/envlist.c @@ -109,9 +109,10 @@ envlist_parse(envlist_t *envlist, const char *env, envvar = strtok_r(tmpenv, ",", &envsave); while (envvar != NULL) { - if ((*callback)(envlist, envvar) != 0) { + int err; + if ((err = (*callback)(envlist, envvar)) != 0) { free(tmpenv); - return (errno); + return (err); } envvar = strtok_r(NULL, ",", &envsave); }
Signed-off-by: Thomas Schwinge <thomas@codesourcery.com> --- util/envlist.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-)