]> xenbits.xensource.com Git - libvirt.git/commitdiff
virNetServerClientNewPostExecRestart: Drop useless typecasts
authorMichal Privoznik <mprivozn@redhat.com>
Thu, 5 May 2016 06:42:50 +0000 (08:42 +0200)
committerMichal Privoznik <mprivozn@redhat.com>
Thu, 5 May 2016 11:48:53 +0000 (13:48 +0200)
In this function, @id is defined as unsigned long long. When
passing this variable to virJSONValueObjectGetNumberUlong(),
well address of this variable, it's typecasted to ull*. There
is no need for that. It's a same story with @nrequests_max.

Signed-off-by: Michal Privoznik <mprivozn@redhat.com>
src/rpc/virnetserverclient.c

index d38f42140d5b0c5d4255fadefbce457c7a4bcc00..d3a3a18f932b0956f778f37630649c75190f7416 100644 (file)
@@ -485,7 +485,7 @@ virNetServerClientPtr virNetServerClientNewPostExecRestart(virJSONValuePtr objec
         return NULL;
     }
     if (virJSONValueObjectGetNumberUint(object, "nrequests_max",
-                                        (unsigned int *)&nrequests_max) < 0) {
+                                        &nrequests_max) < 0) {
         virReportError(VIR_ERR_INTERNAL_ERROR, "%s",
                        _("Missing nrequests_client_max field in JSON state document"));
         return NULL;
@@ -501,8 +501,7 @@ virNetServerClientPtr virNetServerClientNewPostExecRestart(virJSONValuePtr objec
         /* no ID found in, a new one must be generated */
         id = virNetServerNextClientID((virNetServerPtr) opaque);
     } else {
-        if (virJSONValueObjectGetNumberUlong(object, "id",
-                                        (unsigned long long *) &id) < 0) {
+        if (virJSONValueObjectGetNumberUlong(object, "id", &id) < 0) {
         virReportError(VIR_ERR_INTERNAL_ERROR, "%s",
                        _("Malformed id field in JSON state document"));
         return NULL;