Summary: | GeoDataLatLonBox default ctor is little intuitive | ||
---|---|---|---|
Product: | [Applications] marble | Reporter: | Dennis Nienhüser <nienhueser> |
Component: | general | Assignee: | marble-bugs |
Status: | RESOLVED FIXED | ||
Severity: | normal | ||
Priority: | NOR | ||
Version: | unspecified | ||
Target Milestone: | --- | ||
Platform: | unspecified | ||
OS: | Linux | ||
Latest Commit: | http://commits.kde.org/marble/d641bde789a91ca5a296cecebc6fd39f99a2425d | Version Fixed In: |
Description
Dennis Nienhüser
2012-05-13 20:47:05 UTC
Git commit 7bfbed43730a7a840ab3156fe219ab6eb05c4085 by Dennis Nienhüser. Committed on 13/05/2012 at 22:49. Pushed by nienhueser into branch 'master'. Fix bounds calculation M +1 -1 src/lib/routing/Route.cpp http://commits.kde.org/marble/7bfbed43730a7a840ab3156fe219ab6eb05c4085 Git commit b4db6767d8e4d6b88a7445c07e8fa1890303d021 by Dennis Nienhüser. Committed on 20/05/2012 at 10:14. Pushed by nienhueser into branch 'master'. Fix unification with bboxes spanning the entire longitudinal range. M +2 -2 src/lib/geodata/data/GeoDataLatLonBox.cpp M +3 -0 tests/TestGeoDataLatLonAltBox.cpp http://commits.kde.org/marble/b4db6767d8e4d6b88a7445c07e8fa1890303d021 Git commit b296eb97811c57be01eacfbbe7d6bf98447d4064 by Dennis Nienhüser. Committed on 20/05/2012 at 12:45. Pushed by nienhueser into branch 'master'. Handle viewport IDL crossing when calculating visible items. Related: bug 298498 Note that commit 635dc5db mentioned in bug 298498#c0 did not cause the bug itself, but only revealed it: Previously the bounding box was only set in setLineString(), but not in the ctor which took the linestring as well. Hence when using the ctor alone (which is what we do), the bbox was not set to the correct one, but an empty one. Now the default ctor of the bbox returns a sphere sized bbox, hence seemingly making it work (hiding both the bbox calculation bug as well as the negative performance implications). To me that's another argument to change the default ctor of GeoDataLatLon*Box. M +17 -0 src/lib/GeoGraphicsScene.cpp http://commits.kde.org/marble/b296eb97811c57be01eacfbbe7d6bf98447d4064 Git commit d641bde789a91ca5a296cecebc6fd39f99a2425d by Dennis Nienhüser. Committed on 25/05/2012 at 09:38. Pushed by nienhueser into branch 'master'. Have GeoDataLatLonBox(void) create empty boxes. REVIEW: 105038 M +18 -6 src/lib/geodata/data/GeoDataLatLonBox.cpp M +1 -1 src/lib/geodata/data/GeoDataLatLonBox.h M +28 -0 tests/TestGeoDataLatLonAltBox.cpp http://commits.kde.org/marble/d641bde789a91ca5a296cecebc6fd39f99a2425d Git commit 216dc034a916a1ec5509f5c040e0c3eff02ca80d by Dennis Nienhüser. Committed on 25/06/2012 at 20:13. Pushed by nienhueser into branch 'master'. Fix visibility of points, images and ground overlays in the geoscene. CCMAIL: e.konrad@mpegcode.com M +5 -0 src/lib/geodata/graphicsitem/GeoImageGraphicsItem.cpp M +3 -0 src/lib/geodata/graphicsitem/GeoImageGraphicsItem.h M +5 -0 src/lib/geodata/graphicsitem/GeoPhotoGraphicsItem.cpp M +3 -0 src/lib/geodata/graphicsitem/GeoPhotoGraphicsItem.h M +5 -0 src/lib/geodata/graphicsitem/GeoPointGraphicsItem.cpp M +3 -0 src/lib/geodata/graphicsitem/GeoPointGraphicsItem.h http://commits.kde.org/marble/216dc034a916a1ec5509f5c040e0c3eff02ca80d Git commit 2dd24ffff10a3fc572deb6a01cf32519d9307f47 by Dennis Nienhüser. Committed on 25/06/2012 at 20:13. Pushed by nienhueser into branch 'sok-2012-plasma-active'. Fix visibility of points, images and ground overlays in the geoscene. CCMAIL: e.konrad@mpegcode.com M +5 -0 src/lib/geodata/graphicsitem/GeoImageGraphicsItem.cpp M +3 -0 src/lib/geodata/graphicsitem/GeoImageGraphicsItem.h M +5 -0 src/lib/geodata/graphicsitem/GeoPhotoGraphicsItem.cpp M +3 -0 src/lib/geodata/graphicsitem/GeoPhotoGraphicsItem.h M +5 -0 src/lib/geodata/graphicsitem/GeoPointGraphicsItem.cpp M +3 -0 src/lib/geodata/graphicsitem/GeoPointGraphicsItem.h http://commits.kde.org/marble/2dd24ffff10a3fc572deb6a01cf32519d9307f47 |