.appveyor.yml

@@ -1,4 +1,4 @@
-version: 11.4.{build}
+version: 11.3.{build}
 
 configuration:
   - Release

gpxsee.pro

@@ -3,7 +3,7 @@ unix:!macx:!android {
 } else {
     TARGET = GPXSee
 }
-VERSION = 11.4
+VERSION = 11.3
 
 QT += core \
     gui \

pkg/gpxsee64.nsi

@@ -37,7 +37,7 @@ Unicode true
 ; The name of the installer
 Name "GPXSee"
 ; Program version
-!define VERSION "11.4"
+!define VERSION "11.3"
 
 ; The file to write
 OutFile "GPXSee-${VERSION}_x64.exe"

src/map/oruxmap.cpp

@@ -234,10 +234,6 @@ void OruxMap::calibrationPoints(QXmlStreamReader &reader, const QSize &size,
 				return;
 
 			CalibrationPoint p(corner2point(corner, size), Coordinates(lon, lat));
-			if (!p.isValid()) {
-				reader.raiseError(QString("invalid calibration point"));
-				return;
-			}
 			points.append(p);
 
 			reader.readElementText();
@@ -246,15 +242,13 @@ void OruxMap::calibrationPoints(QXmlStreamReader &reader, const QSize &size,
 	}
 }
 
-void OruxMap::mapCalibration(QXmlStreamReader &reader, const QString &dir,
-  int level)
+void OruxMap::mapCalibration(QXmlStreamReader &reader, int level)
 {
 	int zoom;
 	QSize tileSize, size, calibrationSize;
-	QString datum, projection, fileName;
+	QString datum, projection;
 	QList<CalibrationPoint> points;
 	Projection proj;
-	Transform t;
 
 	QXmlStreamAttributes attr = reader.attributes();
 	if (!intAttr(reader, attr, "layerLevel", zoom))
@@ -262,10 +256,10 @@ void OruxMap::mapCalibration(QXmlStreamReader &reader, const QString &dir,
 
 	while (reader.readNextStartElement()) {
 		if (reader.name() == QLatin1String("OruxTracker"))
-			oruxTracker(reader, dir, level + 1);
+			oruxTracker(reader, level + 1);
 		else if (reader.name() == QLatin1String("MapName")) {
 			QString name(reader.readElementText());
-			if (!level && dir.isEmpty())
+			if (!level)
 				_name = name;
 		} else if (reader.name() == QLatin1String("MapChunks")) {
 			int xMax, yMax, width, height;
@@ -283,8 +277,6 @@ void OruxMap::mapCalibration(QXmlStreamReader &reader, const QString &dir,
 				return;
 			if (!strAttr(reader, attr, "projection", projection))
 				return;
-			if (!strAttr(reader, attr, "file_name", fileName))
-				return;
 
 			tileSize = QSize(width, height);
 			size = QSize(xMax * width, yMax * height);
@@ -317,50 +309,29 @@ void OruxMap::mapCalibration(QXmlStreamReader &reader, const QString &dir,
 			calibrationSize = QSize(width, height);
 
 			reader.readElementText();
-		} else if (reader.name() == QLatin1String("CalibrationPoints")) {
+		} else if (reader.name() == QLatin1String("CalibrationPoints"))
 			calibrationPoints(reader, calibrationSize, points);
-
-			t = computeTransformation(proj, points);
-			if (!t.isValid()) {
-				reader.raiseError(t.errorString());
-				return;
-			}
-		} else
+		else
 			reader.skipCurrentElement();
 	}
 
 	if (tileSize.isValid()) {
-		if (!t.isValid()) {
-			reader.raiseError("Invalid map calibration");
-			return;
-		}
-
-		QDir mapDir(QFileInfo(path()).absoluteDir());
-		QDir subDir = dir.isEmpty()
-		  ? mapDir : QDir(mapDir.absoluteFilePath(dir));
-		QString set(subDir.absoluteFilePath("set"));
-
-		_zooms.append(Zoom(zoom, tileSize, size, proj, t, fileName, set));
+		Transform t(computeTransformation(proj, points));
+		_zooms.append(Zoom(zoom, tileSize, size, proj, t));
 	}
 }
 
-void OruxMap::oruxTracker(QXmlStreamReader &reader, const QString &dir,
-  int level)
+void OruxMap::oruxTracker(QXmlStreamReader &reader, int level)
 {
-	if (level > 1 || (level && !dir.isEmpty())) {
-		reader.raiseError("invalid map nesting");
-		return;
-	}
-
 	while (reader.readNextStartElement()) {
 		if (reader.name() == QLatin1String("MapCalibration"))
-			mapCalibration(reader, dir, level);
+			mapCalibration(reader, level);
 		else
 			reader.skipCurrentElement();
 	}
 }
 
-bool OruxMap::readXML(const QString &path, const QString &dir)
+bool OruxMap::readXML(const QString &path)
 {
 	QFile file(path);
 
@@ -370,7 +341,7 @@ bool OruxMap::readXML(const QString &path, const QString &dir)
 	QXmlStreamReader reader(&file);
 	if (reader.readNextStartElement()) {
 		if (reader.name() == QLatin1String("OruxTracker"))
-			oruxTracker(reader, dir, 0);
+			oruxTracker(reader, 0);
 		else
 			reader.raiseError("Not a Orux map calibration file");
 	}
@@ -383,67 +354,51 @@ bool OruxMap::readXML(const QString &path, const QString &dir)
 	return true;
 }
 
+
 OruxMap::OruxMap(const QString &fileName, QObject *parent)
   : Map(fileName, parent), _zoom(0), _mapRatio(1.0), _valid(false)
 {
-	QFileInfo fi(fileName);
-	QDir dir(fi.absoluteDir());
-
 	if (!readXML(fileName))
 		return;
 	if (_zooms.isEmpty()) {
-		QStringList list(dir.entryList(QDir::Dirs | QDir::NoDotAndDotDot));
-		for (int i = 0; i < list.size(); i++) {
-			QDir subDir(dir.absoluteFilePath(list.at(i)));
-			if (!readXML(subDir.absoluteFilePath(list.at(i) + ".otrk2.xml"),
-			  list.at(i))) {
-				_errorString = list.at(i) + ": " + _errorString;
-				return;
-			}
-		}
-		if (_zooms.isEmpty()) {
-			_errorString = "No usable zoom level found";
-			return;
-		}
+		_errorString = "No usable zoom level found";
+		return;
 	}
 	std::sort(_zooms.begin(), _zooms.end());
 
 
-	if (dir.exists("OruxMapsImages.db")) {
-		QString dbFile(dir.absoluteFilePath("OruxMapsImages.db"));
-		_db = QSqlDatabase::addDatabase("QSQLITE", dbFile);
-		_db.setDatabaseName(dbFile);
-		_db.setConnectOptions("QSQLITE_OPEN_READONLY");
-		if (!_db.open()) {
-			_errorString = "Error opening database file";
-			return;
-		}
-
-		QSqlRecord r = _db.record("tiles");
-		if (r.isEmpty()
-		  || r.field(0).name() != "x"
-		  || META_TYPE(r.field(0).type()) != QMetaType::Int
-		  || r.field(1).name() != "y"
-		  || META_TYPE(r.field(1).type()) != QMetaType::Int
-		  || r.field(2).name() != "z"
-		  || META_TYPE(r.field(2).type()) != QMetaType::Int
-		  || r.field(3).name() != "image"
-		  || META_TYPE(r.field(3).type()) != QMetaType::QByteArray) {
-			_errorString = "Invalid table format";
-			return;
-		}
-
-		_db.close();
-	} else {
-		for (int i = 0; i < _zooms.size(); i++) {
-			if (!_zooms.at(i).set.exists()) {
-				_errorString = "missing set directory (level "
-				  + QString::number(_zooms.at(i).zoom) + ")";
-				return;
-			}
-		}
+	QFileInfo fi(fileName);
+	QDir dir(fi.absoluteDir());
+	QString dbFile(dir.absoluteFilePath("OruxMapsImages.db"));
+	if (!QFileInfo::exists(dbFile)) {
+		_errorString = "Image DB file not found";
+		return;
 	}
 
+	_db = QSqlDatabase::addDatabase("QSQLITE", dbFile);
+	_db.setDatabaseName(dbFile);
+	_db.setConnectOptions("QSQLITE_OPEN_READONLY");
+
+	if (!_db.open()) {
+		_errorString = "Error opening database file";
+		return;
+	}
+
+	QSqlRecord r = _db.record("tiles");
+	if (r.isEmpty()
+	  || r.field(0).name() != "x"
+	  || META_TYPE(r.field(0).type()) != QMetaType::Int
+	  || r.field(1).name() != "y"
+	  || META_TYPE(r.field(1).type()) != QMetaType::Int
+	  || r.field(2).name() != "z"
+	  || META_TYPE(r.field(2).type()) != QMetaType::Int
+	  || r.field(3).name() != "image"
+	  || META_TYPE(r.field(3).type()) != QMetaType::QByteArray) {
+		_errorString = "Invalid table format";
+		return;
+	}
+
+	_db.close();
 	_valid = true;
 }
 
@@ -481,14 +436,12 @@ int OruxMap::zoomOut()
 
 void OruxMap::load()
 {
-	if (_db.isValid())
-		_db.open();
+	_db.open();
 }
 
 void OruxMap::unload()
 {
-	if (_db.isValid())
-		_db.close();
+	_db.close();
 }
 
 void OruxMap::setDevicePixelRatio(qreal deviceRatio, qreal mapRatio)
@@ -497,37 +450,19 @@ void OruxMap::setDevicePixelRatio(qreal deviceRatio, qreal mapRatio)
 	_mapRatio = mapRatio;
 }
 
-QPixmap OruxMap::tile(const Zoom &z, int x, int y) const
+QPixmap OruxMap::tile(int zoom, int x, int y) const
 {
-	if (_db.isValid()) {
-		QSqlQuery query(_db);
-		query.prepare("SELECT image FROM tiles WHERE z=:z AND x=:x AND y=:y");
-		query.bindValue(":z", z.zoom);
-		query.bindValue(":x", x);
-		query.bindValue(":y", y);
-		query.exec();
+	QSqlQuery query(_db);
+	query.prepare("SELECT image FROM tiles WHERE z=:z AND x=:x AND y=:y");
+	query.bindValue(":z", zoom);
+	query.bindValue(":x", x);
+	query.bindValue(":y", y);
+	query.exec();
+	if (!query.first())
+		return QPixmap();
 
-		if (!query.first()) {
-			qWarning("%s: SQL %d-%d-%d: not found", qPrintable(name()), z.zoom,
-			  x, y);
-			return QPixmap();
-		} else {
-			QImage img(QImage::fromData(query.value(0).toByteArray()));
-			return QPixmap::fromImage(img);
-		}
-	} else {
-		QString fileName(z.fileName + "_" + QString::number(x) + "_"
-		  + QString::number(y) + ".omc2");
-		QString path(z.set.absoluteFilePath(fileName));
-		if (!QFileInfo::exists(path)) {
-			qWarning("%s: %s: not found", qPrintable(name()),
-			  qPrintable(fileName));
-			return QPixmap();
-		} else {
-			QImage img(path);
-			return QPixmap::fromImage(img);
-		}
-	}
+	QImage img(QImage::fromData(query.value(0).toByteArray()));
+	return QPixmap::fromImage(img);
 }
 
 void OruxMap::draw(QPainter *painter, const QRectF &rect, Flags flags)
@@ -549,12 +484,14 @@ void OruxMap::draw(QPainter *painter, const QRectF &rect, Flags flags)
 			  + "_" + QString::number(x/z.tileSize.width())
 			  + "_" + QString::number(y/z.tileSize.height());
 			if (!QPixmapCache::find(key, &pixmap)) {
-				pixmap = tile(z, x/z.tileSize.width(), y/z.tileSize.height());
+				pixmap = tile(z.zoom, x/z.tileSize.width(), y/z.tileSize.height());
 				if (!pixmap.isNull())
 					QPixmapCache::insert(key, pixmap);
 			}
 
-			if (!pixmap.isNull()) {
+			if (pixmap.isNull())
+				qWarning("%s: error loading tile image", qPrintable(key));
+			else {
 				pixmap.setDevicePixelRatio(_mapRatio);
 				QPointF tp(tl.x() + i * ts.width(), tl.y() + j * ts.height());
 				painter->drawPixmap(tp, pixmap);

src/map/oruxmap.h

@@ -3,7 +3,6 @@
 
 #include <QDebug>
 #include <QSqlDatabase>
-#include <QDir>
 #include "map.h"
 #include "projection.h"
 #include "transform.h"
@@ -45,10 +44,9 @@ public:
 private:
 	struct Zoom {
 		Zoom(int zoom, const QSize &tileSize, const QSize &size,
-		  const Projection &proj, const Transform &transform,
-		  const QString &fileName, const QString &set)
+		  const Projection &proj, const Transform &transform)
 		  : zoom(zoom), tileSize(tileSize), size(size), projection(proj),
-		  transform(transform), fileName(fileName), set(set) {}
+		  transform(transform) {}
 		bool operator<(const Zoom &other) const
 		  {return zoom < other.zoom;}
 
@@ -57,16 +55,14 @@ private:
 		QSize size;
 		Projection projection;
 		Transform transform;
-		QString fileName;
-		QDir set;
 	};
 
-	bool readXML(const QString &path, const QString &dir = QString());
-	void oruxTracker(QXmlStreamReader &reader, const QString &dir, int level);
-	void mapCalibration(QXmlStreamReader &reader, const QString &dir, int level);
+	bool readXML(const QString &path);
+	void oruxTracker(QXmlStreamReader &reader, int level);
+	void mapCalibration(QXmlStreamReader &reader, int level);
 	void calibrationPoints(QXmlStreamReader &reader, const QSize &size,
 	  QList<CalibrationPoint> &points);
-	QPixmap tile(const Zoom &z, int x, int y) const;
+	QPixmap tile(int zoom, int x, int y) const;
 
 	friend QDebug operator<<(QDebug dbg, const Zoom &zoom);