[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v8 01/10] error: auto propagated local_err
|
From: |
Alberto Garcia |
|
Subject: |
Re: [PATCH v8 01/10] error: auto propagated local_err |
|
Date: |
Fri, 06 Mar 2020 12:02:22 +0100 |
|
User-agent: |
Notmuch/0.18.2 (http://notmuchmail.org) Emacs/24.4.1 (i586-pc-linux-gnu) |
On Fri 06 Mar 2020 06:15:27 AM CET, Vladimir Sementsov-Ogievskiy wrote:
Sorry I just gave a quick look at these patches and noticed this:
> + * Function may use error system to return errors. In this case function
> + * defines Error **errp parameter, which should be the last one (except for
> + * functions which varidic argument list), which has the following API:
I don't think that "functions which varidic argument list" is correct
English. Perhaps something like "except for variadic functions" or
"except for functions with a variable number of arguments".
Berto
Re: [PATCH v8 02/10] scripts: add coccinelle script to use auto propagated errp, Markus Armbruster, 2020/03/11
[PATCH v8 01/10] error: auto propagated local_err, Vladimir Sementsov-Ogievskiy, 2020/03/06
[PATCH v8 10/10] xen: introduce ERRP_AUTO_PROPAGATE, Vladimir Sementsov-Ogievskiy, 2020/03/06
Re: [PATCH v8 00/10] error: auto propagated local_err part I, Markus Armbruster, 2020/03/06