You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@openoffice.apache.org by al...@apache.org on 2012/08/16 11:59:55 UTC

svn commit: r1373773 - /incubator/ooo/trunk/main/basctl/source/basicide/baside3.cxx

Author: alg
Date: Thu Aug 16 09:59:54 2012
New Revision: 1373773

URL: http://svn.apache.org/viewvc?rev=1373773&view=rev
Log:
#120515# fixed memory leak

Modified:
    incubator/ooo/trunk/main/basctl/source/basicide/baside3.cxx

Modified: incubator/ooo/trunk/main/basctl/source/basicide/baside3.cxx
URL: http://svn.apache.org/viewvc/incubator/ooo/trunk/main/basctl/source/basicide/baside3.cxx?rev=1373773&r1=1373772&r2=1373773&view=diff
==============================================================================
--- incubator/ooo/trunk/main/basctl/source/basicide/baside3.cxx (original)
+++ incubator/ooo/trunk/main/basctl/source/basicide/baside3.cxx Thu Aug 16 09:59:54 2012
@@ -265,7 +265,10 @@ void DialogWindow::Command( const Comman
 
 IMPL_LINK( DialogWindow, NotifyUndoActionHdl, SfxUndoAction *, pUndoAction )
 {
-	(void)pUndoAction;
+    // #120515# pUndoAction needs to be deleted, this hand over is an ownership
+    // change. As long as it does not get added to the undo manager, it needs at 
+    // least to be deleted.
+	delete pUndoAction;
 
 	// not working yet for unocontrols
 	/*