[Marble-devel] [PATCH 07/13] MergedLayerDecorator: remove unneeded parameters from constructor and paint().
Jens-Michael Hoffmann
jensmh at gmx.de
Tue Mar 30 20:32:52 CEST 2010
MergedLayerDecorator: remove unneeded parameters from constructor and paint().
---
marble/src/lib/MarbleModel.cpp | 4 ++--
marble/src/lib/MergedLayerDecorator.cpp | 5 ++---
marble/src/lib/MergedLayerDecorator.h | 8 ++------
3 files changed, 6 insertions(+), 11 deletions(-)
diff --git a/marble/src/lib/MarbleModel.cpp b/marble/src/lib/MarbleModel.cpp
index 68d65ab..7420c8d 100644
--- a/marble/src/lib/MarbleModel.cpp
+++ b/marble/src/lib/MarbleModel.cpp
@@ -224,7 +224,7 @@ MarbleModel::MarbleModel( QObject *parent )
/* Assume we are dealing with the earth */
d->m_planet = new Planet( "earth" );
d->m_sunLocator = new SunLocator( d->m_dateTime, d->m_planet );
- d->m_layerDecorator = new MergedLayerDecorator( d->m_tileLoader, d->m_sunLocator );
+ d->m_layerDecorator = new MergedLayerDecorator;
connect(d->m_dateTime, SIGNAL( timeChanged() ),
d->m_sunLocator, SLOT( update() ) );
@@ -843,7 +843,7 @@ void MarbleModel::paintTile( StackedTile* tile, GeoSceneTexture *textureLayer )
d->m_layerDecorator->setInfo( tile->id() );
d->m_layerDecorator->setTile( tile->resultTile() );
- d->m_layerDecorator->paint( "maps/" + textureLayer->sourceDir(), mapTheme() );
+ d->m_layerDecorator->paint( "maps/" + textureLayer->sourceDir() );
}
PluginManager* MarbleModel::pluginManager() const
diff --git a/marble/src/lib/MergedLayerDecorator.cpp b/marble/src/lib/MergedLayerDecorator.cpp
index a846d15..f638224 100644
--- a/marble/src/lib/MergedLayerDecorator.cpp
+++ b/marble/src/lib/MergedLayerDecorator.cpp
@@ -25,15 +25,14 @@
using namespace Marble;
-MergedLayerDecorator::MergedLayerDecorator( StackedTileLoader * const tileLoader,
- SunLocator* sunLocator )
+MergedLayerDecorator::MergedLayerDecorator()
: m_tile( 0 ),
m_id(),
m_showTileId( false )
{
}
-void MergedLayerDecorator::paint( const QString& themeId, GeoSceneDocument *mapTheme )
+void MergedLayerDecorator::paint( const QString& themeId )
{
if ( m_showTileId ) {
paintTileId( themeId );
diff --git a/marble/src/lib/MergedLayerDecorator.h b/marble/src/lib/MergedLayerDecorator.h
index af046b3..ab5bf9d 100644
--- a/marble/src/lib/MergedLayerDecorator.h
+++ b/marble/src/lib/MergedLayerDecorator.h
@@ -22,24 +22,20 @@
class QImage;
class QString;
-class QUrl;
namespace Marble
{
-class GeoSceneDocument;
-class SunLocator;
-class StackedTileLoader;
class MergedLayerDecorator : public QObject
{
Q_OBJECT
public:
- MergedLayerDecorator( StackedTileLoader * const tileLoader, SunLocator* sunLocator );
+ MergedLayerDecorator();
// The Parameter themeId is only used for displaying the TileId,
// which is a debugging feature, therefore at this point QString remains.
- void paint( const QString& themeId, GeoSceneDocument *mapTheme = 0 );
+ void paint( const QString& themeId );
void paintTileId(const QString& themeId);
void setShowTileId(bool show);
--
1.7.0.3
More information about the Marble-devel
mailing list