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

[Jens-Michael Hoffmann]

SVN commit 1005225 by jmhoffmann:

Make the following methods const:
- GeoGraphicsItemPrivate::positions()
- LabelGraphicsItemPrivate::font()
- ScreenGraphicsItemPrivate::positions()
- ScreenGraphicsItemPrivate::positivePosition()
- ScreenGraphicsItemPrivate::isMovable()
- GeoGraphicsItem::target()
- GeoGraphicsItem::flags()
- GeoGraphicsItem::positions()
- MarbleGraphicsItemPrivate::positions()
- MarbleGraphicsItemPrivate::boundingRects()


 M  +3 -3      GeoGraphicsItem.cpp  
 M  +3 -3      GeoGraphicsItem.h  
 M  +1 -1      GeoGraphicsItem_p.h  
 M  +1 -1      LabelGraphicsItem.cpp  
 M  +1 -1      LabelGraphicsItem_p.h  
 M  +2 -2      MarbleGraphicsItem_p.h  
 M  +3 -3      ScreenGraphicsItem_p.h  


--- trunk/KDE/kdeedu/marble/src/lib/graphicsview/GeoGraphicsItem.cpp #1005224:1005225
@@ -60,7 +60,7 @@
     p()->m_coordinate = point;
 }
 
-QString GeoGraphicsItem::target()
+QString GeoGraphicsItem::target() const
 {
     return p()->m_target;
 }
@@ -70,7 +70,7 @@
     p()->m_target = target;
 }
 
-GeoGraphicsItem::GeoGraphicsItemFlags GeoGraphicsItem::flags()
+GeoGraphicsItem::GeoGraphicsItemFlags GeoGraphicsItem::flags() const
 {
     return p()->m_flags;
 }
@@ -89,7 +89,7 @@
     p()->m_flags = flags;
 }
 
-QList<QPointF> GeoGraphicsItem::positions()
+QList<QPointF> GeoGraphicsItem::positions() const
 {
     return p()->positions();
 }
--- trunk/KDE/kdeedu/marble/src/lib/graphicsview/GeoGraphicsItem.h #1005224:1005225
@@ -74,7 +74,7 @@
     /**
      * Get the target of the item. The target is the current planet string.s
      */
-    QString target();
+    QString target() const;
 
     /**
      * Set the target of the item with @p target.
@@ -85,7 +85,7 @@
      * Get the GeoGraphicItemFlags value that describes which flags are set on
      * this item. @see QFlags
      */
-    GeoGraphicsItemFlags flags();
+    GeoGraphicsItemFlags flags() const;
 
     /**
      * Set or unset a single flag
@@ -102,7 +102,7 @@
     /**
      * Returns all coordinates of the item in view coordinates according to the given projection.
      */
-    QList<QPointF> positions();
+    QList<QPointF> positions() const;
 
     /**
      * Paints the item in item coordinates
--- trunk/KDE/kdeedu/marble/src/lib/graphicsview/GeoGraphicsItem_p.h #1005224:1005225
@@ -29,7 +29,7 @@
     {
     }
 
-    QList<QPointF> positions()
+    QList<QPointF> positions() const
     {
         return m_positions;
     }
--- trunk/KDE/kdeedu/marble/src/lib/graphicsview/LabelGraphicsItem.cpp #1005224:1005225
@@ -27,7 +27,7 @@
 {
 }
 
-QFont LabelGraphicsItemPrivate::font()
+QFont LabelGraphicsItemPrivate::font() const
 {
     return QApplication::font();
 }
--- trunk/KDE/kdeedu/marble/src/lib/graphicsview/LabelGraphicsItem_p.h #1005224:1005225
@@ -24,7 +24,7 @@
     LabelGraphicsItemPrivate();
 
     // TODO: This has to go up to MarbleGraphicsItem
-    QFont font();
+    QFont font() const;
 
     QString m_text;
 };
--- trunk/KDE/kdeedu/marble/src/lib/graphicsview/MarbleGraphicsItem_p.h #1005224:1005225
@@ -72,7 +72,7 @@
         }
     }
 
-    virtual QList<QPointF> positions()
+    virtual QList<QPointF> positions() const
     {
         return QList<QPointF>();
     }
@@ -86,7 +86,7 @@
         }
     }
 
-    QList<QRectF> boundingRects()
+    QList<QRectF> boundingRects() const
     {
         QList<QRectF> list;
 
--- trunk/KDE/kdeedu/marble/src/lib/graphicsview/ScreenGraphicsItem_p.h #1005224:1005225
@@ -39,7 +39,7 @@
     {
     }
 
-    QList<QPointF> positions()
+    QList<QPointF> positions() const
     {
         QList<QPointF> list;
 
@@ -47,7 +47,7 @@
         return list;
     }
 
-    QPointF positivePosition()
+    QPointF positivePosition() const
     {
         if ( !m_parentSize.isValid() ) {
             qDebug() << "Invalid screen size";
@@ -80,7 +80,7 @@
         screenGraphicsItem->changeViewport( viewport );
     }
 
-    bool isMovable()
+    bool isMovable() const
     {
         return ( m_flags & ScreenGraphicsItem::ItemIsMovable ) ? true : false;
     }