Changeset 26322 in vbox for trunk/src/VBox/Frontends/VBoxHeadless/VBoxHeadless.cpp

Timestamp:

Feb 8, 2010 11:08:02 AM (15 years ago)

Author:

vboxsync

Message:

Frontends/VBoxSDL+VBoxHeadless: code cleanup, reorganize code a little.

File:

• trunk/src/VBox/Frontends/VBoxHeadless/VBoxHeadless.cpp (modified) (24 diffs)

trunk/src/VBox/Frontends/VBoxHeadless/VBoxHeadless.cpp

@@ -71 +71 @@
 #define LogError(m,rc) \
     do { \
-        Log (("VBoxHeadless: ERROR: " m " [rc=0x%08X]\n", rc)); \
-        RTPrintf ("%s\n", m); \
+        Log(("VBoxHeadless: ERROR: " m " [rc=0x%08X]\n", rc)); \
+        RTPrintf("%s\n", m); \
     } while (0)
 
@@ -90 +90 @@
 {
 public:
-    StateChangeEvent (MachineState_T state) : mState (state) {}
+    StateChangeEvent(MachineState_T state) : mState(state) {}
 protected:
     void *handler()
     {
-        LogFlow (("VBoxHeadless: StateChangeEvent: %d\n", mState));
+        LogFlow(("VBoxHeadless: StateChangeEvent: %d\n", mState));
         /* post the termination event if the machine has been PoweredDown/Saved/Aborted */
         if (mState < MachineState_Running)
-            gEventQ->postEvent (NULL);
+            gEventQ->postEvent(NULL);
         return 0;
     }
@@ -275 +275 @@
 public:
 
-    ConsoleCallback ()
+    ConsoleCallback()
     {
 #ifndef VBOX_WITH_XPCOM
@@ -302 +302 @@
     VBOX_SCRIPTABLE_DISPATCH_IMPL(IConsoleCallback)
 
-    STDMETHOD(OnMousePointerShapeChange) (BOOL visible, BOOL alpha, ULONG xHot, ULONG yHot,
-                                          ULONG width, ULONG height, BYTE *shape)
-    {
-        return S_OK;
-    }
-
-    STDMETHOD(OnMouseCapabilityChange) (BOOL supportsAbsolute, BOOL needsHostCursor)
+    STDMETHOD(OnMousePointerShapeChange)(BOOL visible, BOOL alpha, ULONG xHot, ULONG yHot,
+                                         ULONG width, ULONG height, BYTE *shape)
+    {
+        return S_OK;
+    }
+
+    STDMETHOD(OnMouseCapabilityChange)(BOOL supportsAbsolute, BOOL needsHostCursor)
     {
         /* Emit absolute mouse event to actually enable the host mouse cursor. */
@@ -328 +328 @@
     }
 
-    STDMETHOD(OnStateChange) (MachineState_T machineState)
-    {
-        gEventQ->postEvent (new StateChangeEvent (machineState));
-        return S_OK;
-    }
-
-    STDMETHOD(OnExtraDataChange) (BSTR key)
+    STDMETHOD(OnStateChange)(MachineState_T machineState)
+    {
+        gEventQ->postEvent(new StateChangeEvent(machineState));
+        return S_OK;
+    }
+
+    STDMETHOD(OnExtraDataChange)(BSTR key)
     {
         return S_OK;
@@ -344 +344 @@
     }
 
-    STDMETHOD(OnNetworkAdapterChange) (INetworkAdapter *aNetworkAdapter)
-    {
-        return S_OK;
-    }
-
-    STDMETHOD(OnSerialPortChange) (ISerialPort *aSerialPort)
-    {
-        return S_OK;
-    }
-
-    STDMETHOD(OnParallelPortChange) (IParallelPort *aParallelPort)
+    STDMETHOD(OnNetworkAdapterChange)(INetworkAdapter *aNetworkAdapter)
+    {
+        return S_OK;
+    }
+
+    STDMETHOD(OnSerialPortChange)(ISerialPort *aSerialPort)
+    {
+        return S_OK;
+    }
+
+    STDMETHOD(OnParallelPortChange)(IParallelPort *aParallelPort)
     {
         return S_OK;
@@ -412 +412 @@
     }
 
-    STDMETHOD(OnUSBDeviceStateChange) (IUSBDevice *aDevice, BOOL aAttached,
+    STDMETHOD(OnUSBDeviceStateChange)(IUSBDevice *aDevice, BOOL aAttached,
                                       IVirtualBoxErrorInfo *aError)
     {
@@ -418 +418 @@
     }
 
-    STDMETHOD(OnSharedFolderChange) (Scope_T aScope)
+    STDMETHOD(OnSharedFolderChange)(Scope_T aScope)
     {
         return S_OK;
@@ -437 +437 @@
     }
 
-    STDMETHOD(OnShowWindow) (ULONG64 *winId)
+    STDMETHOD(OnShowWindow)(ULONG64 *winId)
     {
         /* OnCanShowWindow() always returns FALSE, so this call should never
@@ -457 +457 @@
 
 #ifdef VBOX_WITH_XPCOM
-NS_DECL_CLASSINFO (VirtualBoxCallback)
-NS_IMPL_THREADSAFE_ISUPPORTS1_CI (VirtualBoxCallback, IVirtualBoxCallback)
-NS_DECL_CLASSINFO (ConsoleCallback)
-NS_IMPL_THREADSAFE_ISUPPORTS1_CI (ConsoleCallback, IConsoleCallback)
+NS_DECL_CLASSINFO(VirtualBoxCallback)
+NS_IMPL_THREADSAFE_ISUPPORTS1_CI(VirtualBoxCallback, IVirtualBoxCallback)
+NS_DECL_CLASSINFO(ConsoleCallback)
+NS_IMPL_THREADSAFE_ISUPPORTS1_CI(ConsoleCallback, IConsoleCallback)
 #endif
 
@@ -597 +597 @@
  *  Entry point.
  */
-extern "C" DECLEXPORT (int) TrustedMain (int argc, char **argv, char **envp)
+extern "C" DECLEXPORT(int) TrustedMain(int argc, char **argv, char **envp)
 {
 #ifdef VBOX_WITH_VRDP
@@ -862 +862 @@
         {
             ComPtr <IMachine> m;
-            rc = virtualBox->FindMachine (Bstr (name), m.asOutParam());
-            if (FAILED (rc))
-            {
-                LogError ("Invalid machine name!\n", rc);
-                break;
-            }
-            m->COMGETTER(Id) (id.asOutParam());
-            AssertComRC (rc);
-            if (FAILED (rc))
-                break;
-        }
-
-        Log (("VBoxHeadless: Opening a session with machine (id={%s})...\n",
+            rc = virtualBox->FindMachine(Bstr(name), m.asOutParam());
+            if (FAILED(rc))
+            {
+                LogError("Invalid machine name!\n", rc);
+                break;
+            }
+            m->COMGETTER(Id)(id.asOutParam());
+            AssertComRC(rc);
+            if (FAILED(rc))
+                break;
+        }
+
+        Log(("VBoxHeadless: Opening a session with machine (id={%s})...\n",
               Utf8Str(id).raw()));
 
         // open a session
-        CHECK_ERROR_BREAK(virtualBox, OpenSession (session, id));
+        CHECK_ERROR_BREAK(virtualBox, OpenSession(session, id));
         fSessionOpened = true;
 
         /* get the console */
         ComPtr <IConsole> console;
-        CHECK_ERROR_BREAK(session, COMGETTER (Console) (console.asOutParam()));
+        CHECK_ERROR_BREAK(session, COMGETTER(Console)(console.asOutParam()));
 
         /* get the machine */
         ComPtr <IMachine> machine;
-        CHECK_ERROR_BREAK(console, COMGETTER(Machine) (machine.asOutParam()));
+        CHECK_ERROR_BREAK(console, COMGETTER(Machine)(machine.asOutParam()));
 
         ComPtr <IDisplay> display;
-        CHECK_ERROR_BREAK(console, COMGETTER(Display) (display.asOutParam()));
+        CHECK_ERROR_BREAK(console, COMGETTER(Display)(display.asOutParam()));
 
 #ifdef VBOX_FFMPEG
@@ -923 +923 @@
                              "you wish to use is supported on your system\n", rcc);
             }
-            if (RT_SUCCESS(rrc) && (S_OK == rcc))
+            if (RT_SUCCESS(rrc) && (rcc == S_OK))
             {
                 Log2(("VBoxHeadless: Registering framebuffer\n"));
@@ -952 +952 @@
             }
 #endif /* defined(VBOX_FFMPEG) */
-            VRDPFramebuffer *pVRDPFramebuffer = new VRDPFramebuffer ();
+            VRDPFramebuffer *pVRDPFramebuffer = new VRDPFramebuffer();
             if (!pVRDPFramebuffer)
             {
@@ -1027 +1027 @@
         /* register a callback for machine events */
         {
-            ConsoleCallback *callback = new ConsoleCallback ();
+            ConsoleCallback *callback = new ConsoleCallback();
             callback->AddRef();
-            CHECK_ERROR(console, RegisterCallback (callback));
+            CHECK_ERROR(console, RegisterCallback(callback));
             callback->Release();
-            if (FAILED (rc))
+            if (FAILED(rc))
                 break;
         }
@@ -1040 +1040 @@
         BOOL fVRDPEnabled;
         ComPtr <IVRDPServer> vrdpServer;
-        CHECK_ERROR_BREAK(machine, COMGETTER (VRDPServer) (vrdpServer.asOutParam()));
-        CHECK_ERROR_BREAK(vrdpServer, COMGETTER(Enabled) (&fVRDPEnabled));
+        CHECK_ERROR_BREAK(machine, COMGETTER(VRDPServer)(vrdpServer.asOutParam()));
+        CHECK_ERROR_BREAK(vrdpServer, COMGETTER(Enabled)(&fVRDPEnabled));
 
         if (vrdpEnabled != NULL)
@@ -1062 +1062 @@
         if (fVRDPEnable)
         {
-            Log (("VBoxHeadless: Enabling VRDP server...\n"));
+            Log(("VBoxHeadless: Enabling VRDP server...\n"));
 
             /* set VRDP port if requested by the user */
@@ -1078 +1078 @@
             if (!fVRDPEnabled)
             {
-                CHECK_ERROR_BREAK(vrdpServer, COMSETTER(Enabled) (TRUE));
+                CHECK_ERROR_BREAK(vrdpServer, COMSETTER(Enabled)(TRUE));
             }
         }
@@ -1086 +1086 @@
             if (fVRDPEnabled)
             {
-                CHECK_ERROR_BREAK(vrdpServer, COMSETTER(Enabled) (FALSE));
-            }
-        }
-#endif
-        Log (("VBoxHeadless: Powering up the machine...\n"));
+                CHECK_ERROR_BREAK(vrdpServer, COMSETTER(Enabled)(FALSE));
+            }
+        }
+#endif
+        Log(("VBoxHeadless: Powering up the machine...\n"));
 
         ComPtr <IProgress> progress;
-        CHECK_ERROR_BREAK(console, PowerUp (progress.asOutParam()));
+        CHECK_ERROR_BREAK(console, PowerUp(progress.asOutParam()));
 
         /* wait for result because there can be errors */
-        if (SUCCEEDED(progress->WaitForCompletion (-1)))
+        if (SUCCEEDED(progress->WaitForCompletion(-1)))
         {
             LONG progressRc;
@@ -1120 +1120 @@
 #endif
 
-        Log (("VBoxHeadless: Waiting for PowerDown...\n"));
+        Log(("VBoxHeadless: Waiting for PowerDown...\n"));
 
         Event *e;
 
-        while (gEventQ->waitForEvent (&e) && e)
-          gEventQ->handleEvent (e);
-
-        Log (("VBoxHeadless: event loop has terminated...\n"));
+        while (gEventQ->waitForEvent(&e) && e)
+          gEventQ->handleEvent(e);
+
+        Log(("VBoxHeadless: event loop has terminated...\n"));
 
 #ifdef VBOX_FFMPEG
         if (pFramebuffer)
         {
-            pFramebuffer->Release ();
+            pFramebuffer->Release();
             Log(("Released framebuffer\n"));
             pFramebuffer = NULL;
@@ -1151 +1151 @@
          * unregister the callback we've registered before.
          */
-        Log (("VBoxHeadless: Closing the session...\n"));
+        Log(("VBoxHeadless: Closing the session...\n"));
         session->Close();
     }
@@ -1161 +1161 @@
     com::Shutdown();
 
-    LogFlow (("VBoxHeadless FINISHED.\n"));
+    LogFlow(("VBoxHeadless FINISHED.\n"));
 
     return rc;
@@ -1171 +1171 @@
  * Main entry point.
  */
-int main (int argc, char **argv, char **envp)
+int main(int argc, char **argv, char **envp)
 {
     // initialize VBox Runtime
@@ -1191 +1191 @@
     }
 
-    return TrustedMain (argc, argv, envp);
+    return TrustedMain(argc, argv, envp);
 }
 #endif /* !VBOX_WITH_HARDENING */