[Marble-commits] KDE/kdeedu/marble/src/lib

[Bastian Holst]

SVN commit 998797 by bholst:

Marble PlacemarkManager didn't delete it's default MarbleGeometryModel, because it doesn't take ownership of it.
Now the MarbleGeometryModel is initialized with 0 to prevent this memory leak.
Thanks to mjansen.


 M  +2 -2      PlacemarkManager.cpp  


--- trunk/KDE/kdeedu/marble/src/lib/PlacemarkManager.cpp #998796:998797
@@ -42,7 +42,7 @@
     public:
         PlacemarkManagerPrivate( QObject* parent )
         : m_model( 0 )
-        , m_geomodel( new MarbleGeometryModel() )
+        , m_geomodel( 0 )
         , m_fileViewModel( new FileViewModel(parent ) )
         , m_finalized( true )
         , m_target( QString() )
@@ -159,7 +159,7 @@
     }
 
     // do not set this file if it only contains points
-    if( doc->isVisible() )
+    if( doc->isVisible() && d->m_geomodel )
         d->m_geomodel->setGeoDataRoot( doc );
     emit geoDataDocumentAdded( *doc );
 }