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

Bernhard Beschow bbeschow at cs.tu-berlin.de
Mon Dec 27 20:20:00 CET 2010


SVN commit 1209645 by beschow:

only pass the stacked TileId in TileLoader::tileCompleted(), since the second TileId was unused in the receiver

 M  +3 -5      StackedTileLoader.cpp  
 M  +5 -4      StackedTileLoader.h  
 M  +1 -1      TileLoader.cpp  
 M  +1 -1      TileLoader.h  


--- trunk/KDE/kdeedu/marble/src/lib/StackedTileLoader.cpp #1209644:1209645
@@ -84,8 +84,8 @@
                                       SunLocator * const sunLocator )
     : d( new StackedTileLoaderPrivate( tileLoader, sunLocator ) )
 {
-    connect( d->m_tileLoader, SIGNAL( tileCompleted( TileId, TileId )),
-             SLOT( updateTile( TileId, TileId )));
+    connect( d->m_tileLoader, SIGNAL( tileCompleted( TileId )),
+             SLOT( updateTile( TileId )));
 }
 
 StackedTileLoader::~StackedTileLoader()
@@ -396,10 +396,8 @@
     d->m_tileCache.setMaxCost( kiloBytes * 1024 );
 }
 
-void StackedTileLoader::updateTile( TileId const & stackedTileId, TileId const & tileId )
+void StackedTileLoader::updateTile( TileId const & stackedTileId )
 {
-    Q_UNUSED(tileId);
-
     d->detectMaxTileLevel();
 
     StackedTile * const displayedTile = d->m_tilesOnDisplay.value( stackedTileId, 0 );
--- trunk/KDE/kdeedu/marble/src/lib/StackedTileLoader.h #1209644:1209645
@@ -147,10 +147,6 @@
         void setVolatileCacheLimit( quint64 kiloBytes );
 
         /**
-         */
-        void updateTile( TileId const & stackedTileId, TileId const & tileId );
-
-        /**
          * Effectively triggers a reload of all tiles that are currently in use
          * and clears the tile cache in physical memory.
          */
@@ -163,6 +159,11 @@
          */
         void tileUpdateAvailable();
 
+    private Q_SLOTS:
+        /**
+         */
+        void updateTile( TileId const & stackedTileId );
+
     private:
         Q_DISABLE_COPY( StackedTileLoader )
         QVector<GeoSceneTexture const *>
--- trunk/KDE/kdeedu/marble/src/lib/TileLoader.cpp #1209644:1209645
@@ -163,7 +163,7 @@
 
     tile->setImage( image );
     tile->setLastModified( QDateTime::currentDateTime() );
-    emit tileCompleted( tile->stackedTileId(), id );
+    emit tileCompleted( tile->stackedTileId() );
 }
 
 void TileLoader::updateTextureLayers()
--- trunk/KDE/kdeedu/marble/src/lib/TileLoader.h #1209644:1209645
@@ -60,7 +60,7 @@
 
     // when this signal is emitted, the TileLoader gives up ownership of
     // the corrsponding tile. Might be better to explicitly transfer...
-    void tileCompleted( TileId const & composedTileId, TileId const & baseTileId );
+    void tileCompleted( TileId const & stackedTileId );
 
  private:
     GeoSceneTexture const * findTextureLayer( TileId const & ) const;


More information about the Marble-commits mailing list