From: Markus Armbruster <armbru@redhat.com>
Date: Sat, 1 Mar 2014 07:40:38 +0000 (+0100)
Subject: qapi: Clean up superfluous null check in qapi_dealloc_type_str()
X-Git-Tag: v2.0.0-rc0~43^2~1
X-Git-Url: https://www.infradead.org/git/?a=commitdiff_plain;h=25a7017555f1b4aeb543b5d323ff4afb8f9c5437;p=users%2Fdwmw2%2Fqemu.git

qapi: Clean up superfluous null check in qapi_dealloc_type_str()

Argument can't be null.  No other Visitor method type_str() checks for
null.

Signed-off-by: Markus Armbruster <armbru@redhat.com>
Reviewed-by: Eric Blake <eblake@redhat.com>
Signed-off-by: Luiz Capitulino <lcapitulino@redhat.com>
---

diff --git a/qapi/qapi-dealloc-visitor.c b/qapi/qapi-dealloc-visitor.c
index dc53545fa5..d0ea118fe3 100644
--- a/qapi/qapi-dealloc-visitor.c
+++ b/qapi/qapi-dealloc-visitor.c
@@ -131,9 +131,7 @@ static void qapi_dealloc_end_list(Visitor *v, Error **errp)
 static void qapi_dealloc_type_str(Visitor *v, char **obj, const char *name,
                                   Error **errp)
 {
-    if (obj) {
-        g_free(*obj);
-    }
+    g_free(*obj);
 }
 
 static void qapi_dealloc_type_int(Visitor *v, int64_t *obj, const char *name,