Const correctness for QgsGeometry::Error

qgis · Sep 7, 2018 · 8ba442f · 8ba442f
1 parent a6e0e39
commit 8ba442f
Show file tree

Hide file tree

Showing 3 changed files with 33 additions and 16 deletions.
diff --git a/python/core/auto_generated/geometry/qgsgeometry.sip.in b/python/core/auto_generated/geometry/qgsgeometry.sip.in
@@ -1481,9 +1481,10 @@ by calling `error()` on the returned geometry.
         explicit Error( const QString &m );
 
         Error( const QString &m, const QgsPointXY &p );
-        QString what();
+        QString what() const;
-        QgsPointXY where();
+        QgsPointXY where() const;
-        bool hasWhere();
+        bool hasWhere() const;
+
     };
 
     enum ValidationMethod

diff --git a/src/core/geometry/qgsgeometry.cpp b/src/core/geometry/qgsgeometry.cpp
@@ -3288,3 +3288,18 @@ QDataStream &operator>>( QDataStream &in, QgsGeometry &geometry )
   return in;
 }
 
+
+QString QgsGeometry::Error::what() const
+{
+  return mMessage;
+}
+
+QgsPointXY QgsGeometry::Error::where() const
+{
+  return mLocation;
+}
+
+bool QgsGeometry::Error::hasWhere() const
+{
+  return mHasLocation;
+}
diff --git a/src/core/geometry/qgsgeometry.h b/src/core/geometry/qgsgeometry.h
@@ -1418,27 +1418,28 @@ class CORE_EXPORT QgsGeometry
      */
     class CORE_EXPORT Error
     {
-        QString message;
-        QgsPointXY location;
-        bool hasLocation = false;
-
       public:
         Error()
-          : message( QStringLiteral( "none" ) )
+          : mMessage( QStringLiteral( "none" ) )
         {}
 
         explicit Error( const QString &m )
-          : message( m )
+          : mMessage( m )
         {}
 
         Error( const QString &m, const QgsPointXY &p )
-          : message( m )
+          : mMessage( m )
-          , location( p )
+          , mLocation( p )
-          , hasLocation( true ) {}
+          , mHasLocation( true ) {}
-
+
-        QString what() { return message; }
+        QString what() const;
-        QgsPointXY where() { return location; }
+        QgsPointXY where() const;
-        bool hasWhere() { return hasLocation; }
+        bool hasWhere() const;
+
+      private:
+        QString mMessage;
+        QgsPointXY mLocation;
+        bool mHasLocation = false;
     };
 
     /**