Cleanup: remove accessor functions from MapLocation

Let's face it: this is a value type. No point in having Java-style
getters and setters. Replace by plain old and boring member variables.

Signed-off-by: Berthold Stoeger <bstoeger@mail.tuwien.ac.at>

1 files changed, 4 insertions, 9 deletions

diff --git a/qt-models/maplocationmodel.h b/qt-models/maplocationmodel.h
index 5fdddf0f2..afb7e92ec 100644
--- a/qt-models/maplocationmodel.h
+++ b/qt-models/maplocationmodel.h
@@ -19,9 +19,6 @@ public:
 	explicit MapLocation(struct dive_site *ds, QGeoCoordinate coord, QString name, bool selected);
 
 	QVariant getRole(int role) const;
-	QGeoCoordinate coordinate();
-	void setCoordinate(QGeoCoordinate coord);
-	struct dive_site *divesite();
 
 	enum Roles {
 		RoleDivesite = Qt::UserRole + 1,
@@ -32,12 +29,10 @@ public:
 		RoleIsSelected
 	};
 
-private:
-	struct dive_site *m_ds;
-	QGeoCoordinate m_coordinate;
-	QString m_name;
-public:
-	bool m_selected = false;
+	struct dive_site *divesite;
+	QGeoCoordinate coordinate;
+	QString name;
+	bool selected = false;
 };
 
 class MapLocationModel : public QAbstractListModel