From kde-bugs-dist  Wed Jul 31 22:34:25 2013
From: Thibaut Gridel <tgridel () free ! fr>
Date: Wed, 31 Jul 2013 22:34:25 +0000
To: kde-bugs-dist
Subject: [marble] [Bug 322783] Optimise PlacemarkLayout
Message-Id: <bug-322783-17878-pGzWwVOZQi () http ! bugs ! kde ! org/>
X-MARC-Message: https://marc.info/?l=kde-bugs-dist&m=137531007001327

https://bugs.kde.org/show_bug.cgi?id=322783

Thibaut Gridel <tgridel@free.fr> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|CONFIRMED                   |RESOLVED
         Resolution|---                         |FIXED
      Latest Commit|                            |http://commits.kde.org/marb
                   |                            |le/b532c4bbb057d9120f94205b
                   |                            |ff5605c6535230c5

--- Comment #1 from Thibaut Gridel <tgridel@free.fr> ---
Git commit b532c4bbb057d9120f94205bff5605c6535230c5 by Thibaut Gridel.
Committed on 31/07/2013 at 22:33.
Pushed by tgridel into branch 'master'.

PlacemarkLayout: optimise for label-less placemarks

roomForLabel is indeed the culprit:
- it didn't scale as it wrongly iterated all placemarks
- it isn't needed once we know a placemark doesn't have text

M  +19   -14   src/lib/PlacemarkLayout.cpp

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

-- 
You are receiving this mail because:
You are watching all bug changes.