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'

Reply via email to