[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 9/9] balloon: Eliminate silly QERR_ macros
From: |
Eric Blake |
Subject: |
Re: [Qemu-devel] [PATCH 9/9] balloon: Eliminate silly QERR_ macros |
Date: |
Wed, 14 Jan 2015 06:31:33 -0700 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Thunderbird/31.3.0 |
On 01/13/2015 10:50 AM, Markus Armbruster wrote:
> The QERR_ macros are leftovers from the days of "rich" error objects.
> They're used with error_set() and qerror_report(), and expand into the
> first *two* arguments. This trickiness has become pointless. Clean
> up.
I like your wording in 6/9 better ("Clean this one up"), as it is more
obvious that you are only fixing a subset and not eliminating all such
macros.
>
> Signed-off-by: Markus Armbruster <address@hidden>
> ---
> balloon.c | 6 ++++--
> include/qapi/qmp/qerror.h | 6 ------
> 2 files changed, 4 insertions(+), 8 deletions(-)
>
For this patch, and for all others in the series where I did not
explicitly reply,
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 3/9] hmp: Compile hmp_info_spice() only with CONFIG_SPICE, (continued)
- [Qemu-devel] [PATCH 3/9] hmp: Compile hmp_info_spice() only with CONFIG_SPICE, Markus Armbruster, 2015/01/13
- [Qemu-devel] [PATCH 6/9] qmp: Eliminate silly QERR_COMMAND_NOT_FOUND macro, Markus Armbruster, 2015/01/13
- [Qemu-devel] [PATCH 2/9] qmp hmp: Improve error messages when SPICE is not in use, Markus Armbruster, 2015/01/13
- [Qemu-devel] [PATCH 7/9] balloon: Inline qemu_balloon(), qemu_balloon_status(), Markus Armbruster, 2015/01/13
- [Qemu-devel] [PATCH 8/9] balloon: Factor out common "is balloon active" test, Markus Armbruster, 2015/01/13
- [Qemu-devel] [PATCH 9/9] balloon: Eliminate silly QERR_ macros, Markus Armbruster, 2015/01/13
- Re: [Qemu-devel] [PATCH 9/9] balloon: Eliminate silly QERR_ macros,
Eric Blake <=
- [Qemu-devel] [PATCH 5/9] qmp: Simplify recognition of capability negotiation command, Markus Armbruster, 2015/01/13
- Re: [Qemu-devel] [PATCH 0/9] qmp hmp balloon: Cleanups around error reporting, Gerd Hoffmann, 2015/01/14