[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v3 0/3] coccinelle: Clean up error checks and re
From: |
Markus Armbruster |
Subject: |
Re: [Qemu-devel] [PATCH v3 0/3] coccinelle: Clean up error checks and return value variables |
Date: |
Tue, 14 Jun 2016 19:45:12 +0200 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/24.5 (gnu/linux) |
Markus Armbruster <address@hidden> writes:
> Eduardo Habkost <address@hidden> writes:
>
>> Changes v1 -> v2:
>> * The Coccinelle scripts were simplified by using "when"
>> constraints to detect when a variable is not used elsewhere
>> inside the function.
>> * Added script to remove unnecessary variables for function
>> return value.
>> * Coccinelle scripts added to scripts/coccinelle.
>>
>> Changes v2 -> v3 on patch 2/3:
>> * Remove unused metavariable from script
>> * Do changes only if errp is not touched before the error_setg()
>> call (so we are sure *errp is not set and error_setg() won't
>> abort)
>> * Changes dropped from v2:
>> * block.c:bdrv_create_co_entry()
>> * block.c:bdrv_create_file()
>> * blockdev.c:qmp_blockdev_mirror()
>>
>> Changes v2 -> v3 on patch 3/3:
>> * Not a RFC anymore
>> * Used --keep-comments option
>> * Instead of using:
>> - VAR = E;
>> + return E;
>> use:
>> - VAR =
>> + return
>> E
>> This makes Coccinelle keep the existing formatting on some
>> cases.
>
> Neat trick.
>
>> * Manual fixups
>
> With the commit message of 3/3 amended, series
> Reviewed-by: Markus Armbruster <address@hidden>
>
> My other suggested touch ups are optional. If you don't object, I'll do
> them, and take the result through my error-next branch.
Applied to error-next, thanks!
- [Qemu-devel] [PATCH v3 2/3] error: Remove unnecessary local_err variables, (continued)
Re: [Qemu-devel] [PATCH v3 0/3] coccinelle: Clean up error checks and return value variables, Markus Armbruster, 2016/06/14