Changeset 896 in vbox for trunk/src/VBox/Frontends/VirtualBox/include

Timestamp:

Feb 14, 2007 11:03:34 AM (18 years ago)

Author:

vboxsync

svn:sync-xref-src-repo-rev:

18614

Message:

1803: FE/Qt leaks memory on Windows

It looks like QCursor uses HCURSOR handle “as is” without copying, so it is not correct to destroy WinAPI HCURSOR just after setting cursor to console viewport.
Unfortunately QCursor will not destroy icon under HCURSOR handle during self-destruction so it is necessary:

1. to make VBoxConsoleView class member – HCURSOR handle,
2. to destroy HCURSOR handle after new cursor is set for viewport,
3. to destroy HCURSOR handle if it is not required any more (upon VBoxConsoleView destruction).

File:

• trunk/src/VBox/Frontends/VirtualBox/include/VBoxConsoleView.h (modified) (2 diffs)

trunk/src/VBox/Frontends/VirtualBox/include/VBoxConsoleView.h

@@ -83 +83 @@
 
     void onFullscreenChange (bool on);
-    
+
     void fixModifierState (LONG *codes, uint *count);
 
@@ -206 +206 @@
     VBoxDefs::RenderMode mode;
 
+#if defined(Q_WS_WIN)
+    HCURSOR mAlphaCursor;
+#endif
+
 #if defined (VBOX_GUI_USE_REFRESH_TIMER)
     QPixmap pm;