Andre Poenitz wrote:
On Wed, Apr 16, 2008 at 07:43:08PM -0000, [EMAIL PROTECTED] wrote:
Author: kuemmel
Date: Wed Apr 16 21:43:06 2008
New Revision: 24299
URL: http://www.lyx.org/trac/changeset/24299
Log:
fix some memory leaks
Modified:
URL:
http://www.lyx.org/trac/file/lyx-devel/trunk/src/frontends/qt4/Menus.cpp?rev=24299
==============================================================================
--- lyx-devel/trunk/src/frontends/qt4/Menus.cpp (original)
+++ lyx-devel/trunk/src/frontends/qt4/Menus.cpp Wed Apr 16 21:43:06 2008
@@ -57,6 +57,8 @@
#include <QList>
#include <QMenuBar>
#include <QString>
+
+#include <boost/shared_ptr.hpp>
#include <algorithm>
#include <ostream>
@@ -1187,7 +1189,7 @@
///
MenuDefinition menubar_;
- typedef QMap<GuiView *, QHash<QString, Menu *> > NameMap;
+ typedef QMap<GuiView *, QHash<QString, boost::shared_ptr<Menu> > >
NameMap;
[...]
- d->name_map_[view][name] = menu;
+ d->name_map_[view][name] = boost::shared_ptr<Menu>(menu);
}
Why don't the stored menus simply get a QObject parent?
Andre'
Too simple ;)
but I changed it to your solution.
Peter
