]> xenbits.xensource.com Git - libvirt.git/commitdiff
vboxDumpDisplay: more indentation reducing
authorJán Tomko <jtomko@redhat.com>
Fri, 5 Feb 2016 16:13:25 +0000 (17:13 +0100)
committerJán Tomko <jtomko@redhat.com>
Tue, 9 Feb 2016 09:07:57 +0000 (10:07 +0100)
VRDxEnabled is initialized to false. Put the if (VRDxEnabled)
on the top level to reduce nesting.

src/vbox/vbox_common.c

index 59088f491a0b917772de191dfeb29eb40f2c5db0..a7839de0e8ca37d5b73ca0130682bf8224ae8dda 100644 (file)
@@ -3398,49 +3398,49 @@ vboxDumpDisplay(virDomainDefPtr def, vboxGlobalData *data, IMachine *machine)
     }
 
     gVBoxAPI.UIMachine.GetVRDxServer(machine, &VRDxServer);
-    if (VRDxServer) {
+    if (VRDxServer)
         gVBoxAPI.UIVRDxServer.GetEnabled(VRDxServer, &VRDxEnabled);
-        if (VRDxEnabled) {
 
-            totalPresent++;
+    if (VRDxEnabled) {
 
-            if ((VIR_REALLOC_N(def->graphics, totalPresent) >= 0) &&
-                (VIR_ALLOC(def->graphics[def->ngraphics]) >= 0)) {
-                PRUnichar *netAddressUtf16   = NULL;
-                char      *netAddressUtf8    = NULL;
-                PRBool allowMultiConnection  = PR_FALSE;
-                PRBool reuseSingleConnection = PR_FALSE;
+        totalPresent++;
 
-                gVBoxAPI.UIVRDxServer.GetPorts(data, VRDxServer, def->graphics[def->ngraphics]);
+        if ((VIR_REALLOC_N(def->graphics, totalPresent) >= 0) &&
+            (VIR_ALLOC(def->graphics[def->ngraphics]) >= 0)) {
+            PRUnichar *netAddressUtf16 = NULL;
+            char *netAddressUtf8 = NULL;
+            PRBool allowMultiConnection = PR_FALSE;
+            PRBool reuseSingleConnection = PR_FALSE;
 
-                def->graphics[def->ngraphics]->type = VIR_DOMAIN_GRAPHICS_TYPE_RDP;
+            gVBoxAPI.UIVRDxServer.GetPorts(data, VRDxServer, def->graphics[def->ngraphics]);
 
-                gVBoxAPI.UIVRDxServer.GetNetAddress(data, VRDxServer, &netAddressUtf16);
-                if (netAddressUtf16) {
-                    VBOX_UTF16_TO_UTF8(netAddressUtf16, &netAddressUtf8);
-                    if (STRNEQ(netAddressUtf8, ""))
-                        virDomainGraphicsListenSetAddress(def->graphics[def->ngraphics], 0,
-                                                          netAddressUtf8, -1, true);
-                    VBOX_UTF16_FREE(netAddressUtf16);
-                    VBOX_UTF8_FREE(netAddressUtf8);
-                }
+            def->graphics[def->ngraphics]->type = VIR_DOMAIN_GRAPHICS_TYPE_RDP;
 
-                gVBoxAPI.UIVRDxServer.GetAllowMultiConnection(VRDxServer, &allowMultiConnection);
-                if (allowMultiConnection)
-                    def->graphics[def->ngraphics]->data.rdp.multiUser = true;
+            gVBoxAPI.UIVRDxServer.GetNetAddress(data, VRDxServer, &netAddressUtf16);
+            if (netAddressUtf16) {
+                VBOX_UTF16_TO_UTF8(netAddressUtf16, &netAddressUtf8);
+                if (STRNEQ(netAddressUtf8, ""))
+                    virDomainGraphicsListenSetAddress(def->graphics[def->ngraphics], 0,
+                                                      netAddressUtf8, -1, true);
+                VBOX_UTF16_FREE(netAddressUtf16);
+                VBOX_UTF8_FREE(netAddressUtf8);
+            }
 
-                gVBoxAPI.UIVRDxServer.GetReuseSingleConnection(VRDxServer, &reuseSingleConnection);
-                if (reuseSingleConnection)
-                    def->graphics[def->ngraphics]->data.rdp.replaceUser = true;
+            gVBoxAPI.UIVRDxServer.GetAllowMultiConnection(VRDxServer, &allowMultiConnection);
+            if (allowMultiConnection)
+                def->graphics[def->ngraphics]->data.rdp.multiUser = true;
 
-                def->ngraphics++;
-            } else {
-                virReportOOMError();
-            }
+            gVBoxAPI.UIVRDxServer.GetReuseSingleConnection(VRDxServer, &reuseSingleConnection);
+            if (reuseSingleConnection)
+                def->graphics[def->ngraphics]->data.rdp.replaceUser = true;
+
+            def->ngraphics++;
+        } else {
+            virReportOOMError();
         }
-        VBOX_RELEASE(VRDxServer);
     }
 
+    VBOX_RELEASE(VRDxServer);
     VBOX_UTF8_FREE(valueTypeUtf8);
 }