[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 04/11] qerror: Eliminate QERR_DEVICE_NOT_FOUND
From: |
Eric Blake |
Subject: |
Re: [Qemu-devel] [PATCH 04/11] qerror: Eliminate QERR_DEVICE_NOT_FOUND |
Date: |
Mon, 15 Jun 2015 07:59:38 -0600 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Thunderbird/31.7.0 |
On 06/13/2015 08:20 AM, Markus Armbruster wrote:
> Error classes other than ERROR_CLASS_GENERIC_ERROR should not be used
> in new code. Hiding them in QERR_ macros makes new uses hard to spot.
> Fortunately, there's just one such macro left. Eliminate it with this
> coccinelle semantic patch:
>
> @@
> expression EP, E;
> @@
> -error_set(EP, QERR_DEVICE_NOT_FOUND, E)
> +error_set(EP, ERROR_CLASS_DEVICE_NOT_FOUND, "Device '%s' not found", E)
>
> Signed-off-by: Markus Armbruster <address@hidden>
> ---
> backends/rng-egd.c | 3 ++-
> blockdev-nbd.c | 3 ++-
> blockdev.c | 33 ++++++++++++++++++++++-----------
> hmp.c | 6 ++++--
> include/qapi/qmp/qerror.h | 3 ---
> net/net.c | 6 ++++--
> qdev-monitor.c | 6 ++++--
> qmp.c | 12 ++++++++----
> qom/object.c | 6 ++++--
> ui/input.c | 3 ++-
> 10 files changed, 52 insertions(+), 29 deletions(-)
Plain transformation would be closer to a 1:1 insertion/deletion count.
The larger insertion count is due to reflowing long lines after the
transformation. Does coccinelle do that for you, or do you have to
touch things up manually? But I'm okay with the result.
Reviewed-by: Eric Blake <address@hidden>
--
Eric Blake eblake redhat com +1-919-301-3266
Libvirt virtualization library http://libvirt.org
signature.asc
Description: OpenPGP digital signature
[Qemu-devel] [PATCH 08/11] qerror: Finally unused, clean up, Markus Armbruster, 2015/06/13
[Qemu-devel] [PATCH 11/11] Include monitor/monitor.h exactly where needed, Markus Armbruster, 2015/06/13