Changeset 20630 in vbox for trunk/src/VBox/Main

Timestamp:

Jun 16, 2009 1:55:38 PM (15 years ago)

Author:

vboxsync

Message:

Python: moved waiting on client side, removed main API for event waiting, as making no sense

Location:

trunk/src/VBox/Main

Files:

• VirtualBoxImpl.cpp (modified) (1 diff)
• glue/vboxapi.py (modified) (5 diffs)
• idl/VirtualBox.xidl (modified) (1 diff)
• include/VirtualBoxImpl.h (modified) (1 diff)
• xpcom/server.cpp (modified) (2 diffs)

trunk/src/VBox/Main/VirtualBoxImpl.cpp

@@ -1891 +1891 @@
 }
 
-
-STDMETHODIMP VirtualBox::WaitForEvents(LONG aTimeout)
-{
-#ifdef RT_OS_WINDOWS
-    ReturnComNotImplemented();
-#else
-    extern nsresult XPCOM_waitForEvents(PRInt32 aTimeout);
-
-    return XPCOM_waitForEvents(aTimeout);
-#endif
-}
 
 STDMETHODIMP VirtualBox::WaitForPropertyChange (IN_BSTR /* aWhat */, ULONG /* aTimeout */,

trunk/src/VBox/Main/glue/vboxapi.py

@@ -145 +145 @@
         return d['result']
 
+    def waitForEvents(self, timeout):
+        # not really supported yet
+        pass
+
+    def deinit(self):
+        import pythoncom
+        pythoncom.CoUninitialize()
+        pass
+
 class PlatformXPCOM:
     def __init__(self, params):
@@ -192 +201 @@
         return d['result']
 
+    def waitForEvents(self, timeout):
+        import xpcom
+        xpcom._xpcom.WaitForEvents(timeout)
+
+    def deinit(self):
+        import xpcom
+        xpcom._xpcom.DeinitCOM()
 
 class PlatformWEBSERVICE:
@@ -235 +251 @@
     def createCallback(self, iface, impl, arg):
         raise Exception("no callbacks for webservices")
+
+    def waitForEvents(self, timeout):
+        # Webservices cannot do that
+        pass
+
+    def deinit(self):
+        # should we do something about it?
+        pass
 
 class SessionManager:
@@ -269 +293 @@
 
     def __del__(self):
+        deinit(self)
+
+    def deinit(self):
         if hasattr(self, "vbox"):
             del self.vbox
+        if hasattr(self, "platform"):
+            self.platform.deinit()
 
     def initPerThread(self):
@@ -288 +317 @@
     def createCallback(self, iface, impl, arg):
         return self.platform.createCallback(iface, impl, arg)
+
+    def waitForEvents(self, timeout):
+        return self.platform.waitForEvents(timeout)

trunk/src/VBox/Main/idl/VirtualBox.xidl

@@ -2644 +2644 @@
       <param name="callback" type="IVirtualBoxCallback" dir="in">
         <desc>Callback object to unregister.</desc>
-      </param>
-    </method>
-
-    <method name="waitForEvents">
-      <desc>
-        Blocking wait for events to happen and process them, if any.
-        One may wish to call this method from the main even loop 
-        to wait for acitivity from VirtualBox, such as callback calling.
-      </desc>
-      <param name="timeout" type="long" dir="in">
-        <desc>
-          Wait timeout in milliseconds.
-          Specify -1 for an indefinite wait.
-          Specify 0 for immediate return if no events to process.
-        </desc>
       </param>
     </method>

trunk/src/VBox/Main/include/VirtualBoxImpl.h

@@ -180 +180 @@
     STDMETHOD(UnregisterCallback) (IVirtualBoxCallback *aCallback);
 
-    STDMETHOD(WaitForEvents) (LONG aTimeout);    
     STDMETHOD(WaitForPropertyChange) (IN_BSTR aWhat, ULONG aTimeout,
                                       BSTR *aChanged, BSTR *aValues);

trunk/src/VBox/Main/xpcom/server.cpp

@@ -244 +244 @@
 static PRBool volatile gKeepRunning = PR_TRUE;
 
-nsresult XPCOM_waitForEvents(PRInt32 aTimeout)
-{
-  nsIEventQueue* q = gEventQ;
-  PRBool hasEvents = false;
-  nsresult rc;
-
-  if (!gKeepRunning || !q)
-    return NS_OK;
-
-  rc = q->PendingEvents(&hasEvents);
-  if (NS_FAILED (rc))
-    return NS_ERROR_FAILURE;
-
-  if (hasEvents)
-  {
-    q->ProcessPendingEvents();
-    return NS_OK;
-  }
-  
-  if (aTimeout == 0)
-     return NS_OK;
-
-  PRInt32 fd;
-  fd = q->GetEventQueueSelectFD();
-
-  if (fd < 0 && aTimeout == -1)
-  {
-    /* fallback */
-    PLEvent *pEvent = NULL;
-    rc = q->WaitForEvent(&pEvent);
-    if (NS_SUCCEEDED(rc))
-      q->HandleEvent(pEvent);
-
-    q->ProcessPendingEvents();
-    return NS_OK;
-  }
-
-  /* Cannot perform timed wait otherwise */
-  AssertReturn(fd >= 0, NS_ERROR_FAILURE);
-  
-  fd_set fdsetR, fdsetE;
-  struct timeval tv;
-
-  FD_ZERO(&fdsetR);
-  FD_SET(fd, &fdsetR);
-
-  fdsetE = fdsetR;
-  if (aTimeout > 0)
-  {
-    /* LogRel(("sleep %d\n", aTimeout)); */
-    tv.tv_sec = (PRInt64)aTimeout / 1000;
-    tv.tv_usec = ((PRInt64)aTimeout % 1000) * 1000;
-  }
-
-  /** @todo: What to do for XPCOM platforms w/o select() ? */
-  int n = select(fd + 1, &fdsetR, NULL, &fdsetE, aTimeout < 0 ? NULL : &tv);
-  rc = (n >= 0) ? NS_OK :  NS_ERROR_FAILURE;
-
-  /* process pending events, no matter what */
-  q->ProcessPendingEvents();
-
-  return rc;
-}
-
 /////////////////////////////////////////////////////////////////////////////
 
@@ -776 +712 @@
 
 /**
- * Hhelper function to register self components upon start-up
+ * Helper function to register self components upon start-up
  * of the out-of-proc server.
  */