|
From: | Eric Blake |
Subject: | Re: [Qemu-devel] [PATCH v4 2/9] numa: split out NumaOptions parsing into parse_NumaOptions() |
Date: | Fri, 23 Mar 2018 15:49:38 -0500 |
User-agent: | Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.6.0 |
On 03/23/2018 03:42 PM, Eduardo Habkost wrote:
On Mon, Mar 12, 2018 at 02:11:08PM +0100, Igor Mammedov wrote:it will allow to reuse parse_NumaOptions() for parsing configuration commands received via QMP interface Signed-off-by: Igor Mammedov <address@hidden> ---
end: + if (err) { + error_propagate(errp, err); + }"if (err)" is not necessary here. See scripts/coccinelle/error_propagate_null.cocci.
+ parse_NumaOptions(ms, object, &err); + +end: qapi_free_NumaOptions(object); if (err) { error_report_err(err);We can fix this one too while at it.
Hmm - this is the same script mentioned here: https://lists.gnu.org/archive/html/qemu-devel/2018-03/msg06293.htmlExcept that patch didn't pick up this file. Why is Coccinelle not seeing this?
-- Eric Blake, Principal Software Engineer Red Hat, Inc. +1-919-301-3266 Virtualization: qemu.org | libvirt.org
[Prev in Thread] | Current Thread | [Next in Thread] |