Commit 6f474c99 authored by Andreas Bircher's avatar Andreas Bircher

compile fix

parent ec93ab96
...@@ -41,7 +41,7 @@ void TerrainBatchManager::addQuery(ElevationProvider* elevationProvider, const Q ...@@ -41,7 +41,7 @@ void TerrainBatchManager::addQuery(ElevationProvider* elevationProvider, const Q
} }
qCDebug(ElevationProviderLog) << "All altitudes taken from cached data"; qCDebug(ElevationProviderLog) << "All altitudes taken from cached data";
elevationProvider->_signalTerrainData(true, altitudes); elevationProvider->_signalTerrainData(coordinates.count() == altitudes.count(), altitudes);
} }
} }
...@@ -71,6 +71,7 @@ bool TerrainBatchManager::_getAltitudesForCoordinates(const QList<QGeoCoordinate ...@@ -71,6 +71,7 @@ bool TerrainBatchManager::_getAltitudesForCoordinates(const QList<QGeoCoordinate
_tileDownloadQueue.append(tileHash); _tileDownloadQueue.append(tileHash);
} }
_tilesMutex.unlock();
return false; return false;
} else { } else {
...@@ -123,11 +124,13 @@ void TerrainBatchManager::_fetchedTile() ...@@ -123,11 +124,13 @@ void TerrainBatchManager::_fetchedTile()
} else { } else {
qCDebug(ElevationProviderLog) << "Elevation tile fetching returned error. " << reply->errorString(); qCDebug(ElevationProviderLog) << "Elevation tile fetching returned error. " << reply->errorString();
} }
_tileFailed();
reply->deleteLater(); reply->deleteLater();
return; return;
} }
if (!reply->isFinished()) { if (!reply->isFinished()) {
qCDebug(ElevationProviderLog) << "Error in fetching elevation tile. Not finished. " << reply->errorString(); qCDebug(ElevationProviderLog) << "Error in fetching elevation tile. Not finished. " << reply->errorString();
_tileFailed();
reply->deleteLater(); reply->deleteLater();
return; return;
} }
...@@ -140,6 +143,7 @@ void TerrainBatchManager::_fetchedTile() ...@@ -140,6 +143,7 @@ void TerrainBatchManager::_fetchedTile()
if (parseError.error != QJsonParseError::NoError) { if (parseError.error != QJsonParseError::NoError) {
qCDebug(ElevationProviderLog) << "Could not parse terrain tile " << parseError.errorString(); qCDebug(ElevationProviderLog) << "Could not parse terrain tile " << parseError.errorString();
qCDebug(ElevationProviderLog) << responseBytes; qCDebug(ElevationProviderLog) << responseBytes;
_tileFailed();
reply->deleteLater(); reply->deleteLater();
return; return;
} }
...@@ -162,7 +166,7 @@ void TerrainBatchManager::_fetchedTile() ...@@ -162,7 +166,7 @@ void TerrainBatchManager::_fetchedTile()
for (int i = _requestQueue.count() - 1; i >= 0; i--) { for (int i = _requestQueue.count() - 1; i >= 0; i--) {
QList<float> altitudes; QList<float> altitudes;
if (_getAltitudesForCoordinates(_requestQueue[i].coordinates, altitudes)) { if (_getAltitudesForCoordinates(_requestQueue[i].coordinates, altitudes)) {
_requestQueue[i].elevationProvider->_signalTerrainData(true, altitudes); _requestQueue[i].elevationProvider->_signalTerrainData(_requestQueue[i].coordinates.count() == altitudes.count(), altitudes);
_requestQueue.removeAt(i); _requestQueue.removeAt(i);
} }
} }
...@@ -192,3 +196,8 @@ bool ElevationProvider::queryTerrainData(const QList<QGeoCoordinate>& coordinate ...@@ -192,3 +196,8 @@ bool ElevationProvider::queryTerrainData(const QList<QGeoCoordinate>& coordinate
return false; return false;
} }
void ElevationProvider::_signalTerrainData(bool success, QList<float>& altitudes)
{
emit terrainData(success, altitudes);
}
...@@ -34,7 +34,6 @@ public: ...@@ -34,7 +34,6 @@ public:
void addQuery(ElevationProvider* elevationProvider, const QList<QGeoCoordinate>& coordinates); void addQuery(ElevationProvider* elevationProvider, const QList<QGeoCoordinate>& coordinates);
private slots: private slots:
void _requestFinished (void);
void _fetchedTile (void); /// slot to handle fetched elevation tiles void _fetchedTile (void); /// slot to handle fetched elevation tiles
private: private:
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment