Removed magic number used for undefined values

diff --git a/src/map/maptile.cpp b/src/map/maptile.cpp
index 5872f51..7b3306a 100644 (file)
--- a/src/map/maptile.cpp
+++ b/src/map/maptile.cpp
@@ -29,10 +29,9 @@
 
 MapTile::MapTile()
 {
-    // -1 is for undefined
-    m_tileNumber = QPoint(-1, -1);
-    m_zoomLevel = -1;
-    setPos(-1, -1);
+    m_tileNumber = QPoint(UNDEFINED, UNDEFINED);
+    m_zoomLevel = UNDEFINED;
+    setPos(UNDEFINED, UNDEFINED);
 }
 
 int MapTile::zoomLevel()
@@ -69,7 +68,7 @@ void MapTile::setPosition()
         return; // done
     }
     // else
-    setPos(-1, -1);
+    setPos(UNDEFINED, UNDEFINED);
     //Q_ASSERT_X(false, __PRETTY_FUNCTION__, "m_zoomLevel and/or m_tileNumber is undefined");
 }
 

diff --git a/src/map/maptile.h b/src/map/maptile.h
index 3ae2f9b..884f9a2 100644 (file)
--- a/src/map/maptile.h
+++ b/src/map/maptile.h
@@ -60,9 +60,14 @@ private:
     /// \brief Set position of the tile in the MapScene coordinate system
     ///
     /// Does set the position based on the m_zoomLevel and the m_TileNumber. Position is set to
-    /// (-1, -1) if there is something wrong with zoom level or tile numbers
+    /// (UNDEFINED, UNDEFINED) if there is something wrong with zoom level or tile numbers
     void setPosition();
 
+public:
+    /// \var UNDEFINED
+    /// \brief Value to be used when zoom level, tile numbers or position are not defined
+    static const int UNDEFINED = -1;
+
 private:
     int m_zoomLevel; ///< Zoom level
     QPoint m_tileNumber; ///< Tile number