[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-commits
Subject:    [marble] src/lib/marble: No need to sort the tiles, the list is sorted in the end.
From:       Dennis_Nienhüser <nienhueser () kde ! org>
Date:       2015-10-31 18:58:46
Message-ID: E1ZsbMU-0006my-Vy () scm ! kde ! org
[Download RAW message or body]

Git commit cf455de80d96aed40aa115321dfc8144ed4b8cdf by Dennis Nienhüser.
Committed on 31/10/2015 at 18:28.
Pushed by nienhueser into branch 'master'.

No need to sort the tiles, the list is sorted in the end.

M  +1    -3    src/lib/marble/PlacemarkLayout.cpp

http://commits.kde.org/marble/cf455de80d96aed40aa115321dfc8144ed4b8cdf

diff --git a/src/lib/marble/PlacemarkLayout.cpp b/src/lib/marble/PlacemarkLayout.cpp
index 6e28148..4930166 100644
--- a/src/lib/marble/PlacemarkLayout.cpp
+++ b/src/lib/marble/PlacemarkLayout.cpp
@@ -459,10 +459,8 @@ QVector<VisiblePlacemark *> PlacemarkLayout::generateLayout( const ViewportParam
 
     const QItemSelection selection = m_selectionModel->selection();
 
-    QList<TileId> tileIdList = visibleTiles( viewport ).toList();
-    qSort( tileIdList );
     QList<const GeoDataPlacemark*> placemarkList;
-    foreach ( const TileId &tileId, tileIdList ) {
+    foreach ( const TileId &tileId, visibleTiles( viewport ) ) {
         placemarkList += m_placemarkCache.value( tileId );
     }
     qSort(placemarkList.begin(), placemarkList.end(), GeoDataPlacemark::placemarkLayoutOrderCompare);

[prev in list] [next in list] [prev in thread] [next in thread]