Changeset 3884 in vbox for trunk/src/VBox/Additions

Timestamp:

Jul 26, 2007 3:58:26 PM (18 years ago)

Author:

vboxsync

Message:

Filter out duplicates

File:

• trunk/src/VBox/Additions/WINNT/VBoxService/VBoxSeamless.cpp (modified) (4 diffs)

trunk/src/VBox/Additions/WINNT/VBoxService/VBoxSeamless.cpp

@@ -35 +35 @@
     const VBOXSERVICEENV *pEnv;
 
-    HMODULE hModule;
-
-    BOOL (* pfnVBoxInstallHook)(HMODULE hDll, HWND hwndPostWindow);
-    BOOL (* pfnVBoxRemoveHook)();
-
+    HMODULE    hModule;
+
+    BOOL    (* pfnVBoxInstallHook)(HMODULE hDll, HWND hwndPostWindow);
+    BOOL    (* pfnVBoxRemoveHook)();
+
+    LPRGNDATA lpRgnData;
 } VBOXSEAMLESSCONTEXT;
 
@@ -132 +133 @@
     if (gCtx.pfnVBoxRemoveHook)
         gCtx.pfnVBoxRemoveHook();
+
+    if (gCtx.lpRgnData)
+    {
+        free(gCtx.lpRgnData);
+        gCtx.lpRgnData = NULL;
+    }
 }
 
@@ -221 +228 @@
         if (cbSize)
         {
-            LPRGNDATA lpRgnData = (LPRGNDATA)HeapAlloc(GetProcessHeap(), HEAP_ZERO_MEMORY, cbSize);
-
+            LPRGNDATA lpRgnData = (LPRGNDATA)malloc(cbSize);
+            memset(lpRgnData, 0, cbSize);
             if (lpRgnData)
             {
@@ -237 +244 @@
                     }
 #endif
-                    /* send to display driver */
-                    ExtEscape(param.hdc, VBOXESC_SETVISIBLEREGION, cbSize, (LPCSTR)lpRgnData, 0, NULL);
+                    if (    !gCtx.lpRgnData 
+                        ||  lpRgnData->rdh.dwSize != gCtx.lpRgnData->rdh.dwSize
+                        ||  memcmp(gCtx.lpRgnData, lpRgnData, lpRgnData->rdh.dwSize))
+                    {
+                        /* send to display driver */
+                        ExtEscape(param.hdc, VBOXESC_SETVISIBLEREGION, cbSize, (LPCSTR)lpRgnData, 0, NULL);
+
+                        if (gCtx.lpRgnData)
+                            free(gCtx.lpRgnData);
+                        gCtx.lpRgnData = lpRgnData;
+                    }
+                    else
+                        dprintf(("Visible rectangles haven't changed; ignore\n"));
                 }
-                HeapFree(GetProcessHeap(), 0, lpRgnData);
+                if (lpRgnData != gCtx.lpRgnData)
+                    free(lpRgnData);
             }
         }