.appveyor.yml

@@ -1,4 +1,4 @@
-version: 11.1.{build}
+version: 11.0.{build}
 
 configuration:
   - Release

gpxsee.pro

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

pkg/gpxsee64.nsi

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

src/GUI/gui.cpp

@@ -2477,8 +2477,8 @@ void GUI::writeSettings()
 		settings.setValue(SHOW_TICKS_SETTING,
 		  _showTicksAction->isChecked());
 	bool sm = _showMarkersAction->isChecked()
-	  || _showMarkerDateAction->isChecked()
-	  || _showMarkerCoordinatesAction->isChecked();
+	  | _showMarkerDateAction->isChecked()
+	  | _showMarkerCoordinatesAction->isChecked();
 	if (sm != SHOW_MARKERS_DEFAULT)
 		settings.setValue(SHOW_MARKERS_SETTING, sm);
 	if (_showMarkerDateAction->isChecked()

src/GUI/mapview.cpp

@@ -663,8 +663,7 @@ void MapView::wheelEvent(QWheelEvent *event)
 	zoom((delta > 0) ? 1 : -1, event->position().toPoint(), shift);
 #endif // QT 5.15
 
-	/* Do not call QGraphicsView::wheelEvent() here as this would shift the
-	   view ! */
+	QGraphicsView::wheelEvent(event);
 }
 
 void MapView::keyPressEvent(QKeyEvent *event)

src/GUI/navigationwidget.cpp

@@ -1,8 +1,7 @@
+#include "navigationwidget.h"
 #include <QEvent>
 #include <QResizeEvent>
 #include <QPainter>
-#include "mapview.h"
-#include "navigationwidget.h"
 
 #define MARGIN 5
 #define SIZE   40

src/GUI/navigationwidget.h

@@ -2,8 +2,7 @@
 #define NAVIGATIONWIDGET_H
 
 #include <QWidget>
-
-class MapView;
+#include "mapview.h"
 
 #ifdef Q_OS_ANDROID
 class NavigationWidget : public QWidget

src/GUI/optionsdialog.cpp

@@ -717,7 +717,7 @@ QWidget *OptionsDialog::createSystemPage()
 
 	_pixmapCache = new QSpinBox();
 	_pixmapCache->setMinimum(16);
-	_pixmapCache->setMaximum(2048);
+	_pixmapCache->setMaximum(1024);
 	_pixmapCache->setSuffix(UNIT_SPACE + tr("MB"));
 	_pixmapCache->setValue(_options.pixmapCache);
 

src/GUI/settings.h

@@ -217,11 +217,7 @@
 #define ENABLE_HTTP2_SETTING              "enableHTTP2"
 #define ENABLE_HTTP2_DEFAULT              true
 #define PIXMAP_CACHE_SETTING              "pixmapCache"
-#ifdef Q_OS_ANDROID
 #define PIXMAP_CACHE_DEFAULT              256 /* MB */
-#else // Q_OS_ANDROID
-#define PIXMAP_CACHE_DEFAULT              512 /* MB */
-#endif // Q_OS_ANDROID
 #define CONNECTION_TIMEOUT_SETTING        "connectionTimeout"
 #define CONNECTION_TIMEOUT_DEFAULT        30 /* s */
 #define HIRES_PRINT_SETTING               "hiresPrint"

src/map/IMG/rastertile.cpp

@@ -174,8 +174,6 @@ void RasterTile::render()
 
 	qDeleteAll(textItems);
 
-	_valid = true;
-
 	//painter.setPen(Qt::red);
 	//painter.setRenderHint(QPainter::Antialiasing, false);
 	//painter.drawRect(QRect(_xy, _pixmap.size()));
@@ -187,7 +185,7 @@ void RasterTile::ll2xy(QList<MapData::Poly> &polys)
 		MapData::Poly &poly = polys[i];
 		for (int j = 0; j < poly.points.size(); j++) {
 			QPointF &p = poly.points[j];
-			p = ll2xy(Coordinates(p.x(), p.y()));
+			p = _map->ll2xy(Coordinates(p.x(), p.y()));
 		}
 	}
 }
@@ -195,7 +193,7 @@ void RasterTile::ll2xy(QList<MapData::Poly> &polys)
 void RasterTile::ll2xy(QList<MapData::Point> &points)
 {
 	for (int i = 0; i < points.size(); i++) {
-		QPointF p(ll2xy(points.at(i).coordinates));
+		QPointF p(_map->ll2xy(points.at(i).coordinates));
 		points[i].coordinates = Coordinates(p.x(), p.y());
 	}
 }
@@ -212,8 +210,8 @@ void RasterTile::drawPolygons(QPainter *painter)
 
 			if (poly.raster.isValid()) {
 				RectC r(poly.raster.rect());
-				QPointF tl(ll2xy(r.topLeft()));
-				QPointF br(ll2xy(r.bottomRight()));
+				QPointF tl(_map->ll2xy(r.topLeft()));
+				QPointF br(_map->ll2xy(r.bottomRight()));
 				QSizeF size(QRectF(tl, br).size());
 
 				bool insert = false;

src/map/IMG/rastertile.h

@@ -3,8 +3,6 @@
 
 #include <QPixmap>
 #include "mapdata.h"
-#include "map/projection.h"
-#include "map/transform.h"
 
 class QPainter;
 class IMGMap;
@@ -17,25 +15,20 @@ class Style;
 class RasterTile
 {
 public:
-	RasterTile(const Projection &proj, const Transform &transform,
-	  const Style *style, int zoom, const QRect &rect, qreal ratio,
-	  const QString &key, const QList<MapData::Poly> &polygons,
+	RasterTile(IMGMap *map, const Style *style, int zoom, const QRect &rect,
+	  qreal ratio, const QString &key, const QList<MapData::Poly> &polygons,
 	  const QList<MapData::Poly> &lines, QList<MapData::Point> &points)
-	  : _proj(proj), _transform(transform), _style(style), _zoom(zoom),
-	  _rect(rect), _ratio(ratio), _key(key),
-	  _pixmap(rect.width() * ratio, rect.height() * ratio), _polygons(polygons),
-	  _lines(lines), _points(points), _valid(false) {}
+	  : _map(map), _style(style), _zoom(zoom), _rect(rect), _ratio(ratio),
+	  _key(key), _pixmap(rect.width() * ratio, rect.height() * ratio),
+	  _polygons(polygons), _lines(lines), _points(points) {}
 
 	const QString &key() const {return _key;}
 	QPoint xy() const {return _rect.topLeft();}
 	const QPixmap &pixmap() const {return _pixmap;}
-	bool isValid() const {return _valid;}
 
 	void render();
 
 private:
-	QPointF ll2xy(const Coordinates &c) const
-	  {return _transform.proj2img(_proj.ll2xy(c));}
 	void ll2xy(QList<MapData::Poly> &polys);
 	void ll2xy(QList<MapData::Point> &points);
 
@@ -49,8 +42,7 @@ private:
 	void processShields(QList<TextItem*> &textItems);
 	void processStreetNames(QList<TextItem*> &textItems);
 
-	Projection _proj;
-	Transform _transform;
+	IMGMap *_map;
 	const Style *_style;
 	int _zoom;
 	QRect _rect;
@@ -60,7 +52,6 @@ private:
 	QList<MapData::Poly> _polygons;
 	QList<MapData::Poly> _lines;
 	QList<MapData::Point> _points;
-	bool _valid;
 };
 
 }

src/map/imgmap.cpp

@@ -100,8 +100,6 @@ int IMGMap::zoomFit(const QSize &size, const RectC &rect)
 
 int IMGMap::zoomIn()
 {
-	cancelJobs();
-
 	_zoom = qMin(_zoom + 1, _data.first()->zooms().max());
 	updateTransform();
 	return _zoom;
@@ -109,8 +107,6 @@ int IMGMap::zoomIn()
 
 int IMGMap::zoomOut()
 {
-	cancelJobs();
-
 	_zoom = qMax(_zoom - 1, _data.first()->zooms().min());
 	updateTransform();
 	return _zoom;
@@ -143,55 +139,10 @@ void IMGMap::updateTransform()
 		_bounds.adjust(0.5, 0, -0.5, 0);
 }
 
-bool IMGMap::isRunning(const QString &key) const
-{
-	for (int i = 0; i < _jobs.size(); i++) {
-		const QList<IMG::RasterTile> &tiles = _jobs.at(i)->tiles();
-		for (int j = 0; j < tiles.size(); j++)
-			if (tiles.at(j).key() == key)
-				return true;
-	}
-
-	return false;
-}
-
-void IMGMap::runJob(IMGMapJob *job)
-{
-	_jobs.append(job);
-
-	connect(job, &IMGMapJob::finished, this, &IMGMap::jobFinished);
-	job->run();
-}
-
-void IMGMap::removeJob(IMGMapJob *job)
-{
-	_jobs.removeOne(job);
-	job->deleteLater();
-}
-
-void IMGMap::jobFinished(IMGMapJob *job)
-{
-	const QList<IMG::RasterTile> &tiles = job->tiles();
-
-	for (int i = 0; i < tiles.size(); i++) {
-		const IMG::RasterTile &mt = tiles.at(i);
-		if (mt.isValid())
-			QPixmapCache::insert(mt.key(), mt.pixmap());
-	}
-
-	removeJob(job);
-
-	emit tilesLoaded();
-}
-
-void IMGMap::cancelJobs()
-{
-	for (int i = 0; i < _jobs.size(); i++)
-		_jobs.at(i)->cancel();
-}
-
 void IMGMap::draw(QPainter *painter, const QRectF &rect, Flags flags)
 {
+	Q_UNUSED(flags);
+
 	QPointF tl(floor(rect.left() / TILE_SIZE)
 	  * TILE_SIZE, floor(rect.top() / TILE_SIZE) * TILE_SIZE);
 	QSizeF s(rect.right() - tl.x(), rect.bottom() - tl.y());
@@ -208,9 +159,6 @@ void IMGMap::draw(QPainter *painter, const QRectF &rect, Flags flags)
 				QString key(_data.at(n)->fileName() + "-" + QString::number(_zoom)
 				  + "_" + QString::number(ttl.x()) + "_" + QString::number(ttl.y()));
 
-				if (isRunning(key))
-					continue;
-
 				if (QPixmapCache::find(key, &pm))
 					painter->drawPixmap(ttl, pm);
 				else {
@@ -234,8 +182,7 @@ void IMGMap::draw(QPainter *painter, const QRectF &rect, Flags flags)
 					_data.at(n)->points(pointRectD.toRectC(_projection, 20),
 					  _zoom, &points);
 
-					tiles.append(RasterTile(_projection, _transform,
-					  _data.at(n)->style(), _zoom,
+					tiles.append(RasterTile(this, _data.at(n)->style(), _zoom,
 					  QRect(ttl, QSize(TILE_SIZE, TILE_SIZE)), _tileRatio, key,
 					  polygons, lines, points));
 				}
@@ -243,19 +190,14 @@ void IMGMap::draw(QPainter *painter, const QRectF &rect, Flags flags)
 		}
 	}
 
-	if (!tiles.isEmpty()) {
-		if (flags & Map::Block) {
-			QFuture<void> future = QtConcurrent::map(tiles, &RasterTile::render);
-			future.waitForFinished();
+	QFuture<void> future = QtConcurrent::map(tiles, &RasterTile::render);
+	future.waitForFinished();
 
-			for (int i = 0; i < tiles.size(); i++) {
-				const RasterTile &mt = tiles.at(i);
-				const QPixmap &pm = mt.pixmap();
-				painter->drawPixmap(mt.xy(), pm);
-				QPixmapCache::insert(mt.key(), pm);
-			}
-		} else
-			runJob(new IMGMapJob(tiles));
+	for (int i = 0; i < tiles.size(); i++) {
+		const RasterTile &mt = tiles.at(i);
+		const QPixmap &pm = mt.pixmap();
+		painter->drawPixmap(mt.xy(), pm);
+		QPixmapCache::insert(mt.key(), pm);
 	}
 }
 

src/map/imgmap.h

@@ -1,44 +1,12 @@
 #ifndef IMGMAP_H
 #define IMGMAP_H
 
-#include <QtConcurrent>
 #include "map.h"
 #include "projection.h"
 #include "transform.h"
 #include "IMG/mapdata.h"
-#include "IMG/rastertile.h"
 
 
-class IMGMapJob : public QObject
-{
-	Q_OBJECT
-
-public:
-	IMGMapJob(const QList<IMG::RasterTile> &tiles)
-	  : _tiles(tiles) {}
-
-	void run()
-	{
-		connect(&_watcher, &QFutureWatcher<void>::finished, this,
-		  &IMGMapJob::handleFinished);
-		_future = QtConcurrent::map(_tiles, &IMG::RasterTile::render);
-		_watcher.setFuture(_future);
-	}
-	void cancel() {_future.cancel();}
-	const QList<IMG::RasterTile> &tiles() const {return _tiles;}
-
-signals:
-	void finished(IMGMapJob *job);
-
-private slots:
-	void handleFinished() {emit finished(this);}
-
-private:
-	QFutureWatcher<void> _watcher;
-	QFuture<void> _future;
-	QList<IMG::RasterTile> _tiles;
-};
-
 class IMGMap : public Map
 {
 	Q_OBJECT
@@ -76,16 +44,9 @@ public:
 
 	static Map* create(const QString &path, const Projection &, bool *isDir);
 
-private slots:
-	void jobFinished(IMGMapJob *job);
-
 private:
 	Transform transform(int zoom) const;
 	void updateTransform();
-	bool isRunning(const QString &key) const;
-	void runJob(IMGMapJob *job);
-	void removeJob(IMGMapJob *job);
-	void cancelJobs();
 
 	QList<IMG::MapData *> _data;
 	int _zoom;
@@ -95,8 +56,6 @@ private:
 	RectC _dataBounds;
 	qreal _tileRatio;
 
-	QList<IMGMapJob*> _jobs;
-
 	bool _valid;
 	QString _errorString;
 };

src/map/mapsforgemap.cpp

@@ -1,5 +1,4 @@
 #include <QPainter>
-#include <QPixmapCache>
 #include "common/wgs84.h"
 #include "pcs.h"
 #include "rectd.h"

src/map/mapsforgemap.h

@@ -2,6 +2,7 @@
 #define MAPSFORGEMAP_H
 
 #include <QtConcurrent>
+#include <QPixmapCache>
 #include "mapsforge/mapdata.h"
 #include "mapsforge/rastertile.h"
 #include "projection.h"