The ctor without arguments creates a latlonbox that covers the full globe. Being used to QRect and similar I find this little intuitive and would expect an empty sized box instead. Here are two examples why this is not intuitive: 1) GeoDataLatLonBox one; one.isEmpty(); // true one.isNull(); // false GeoDataLatLonBox two(0,0,0,0); two.isEmpty(); // false two.isNull(); // true In my opinion isNull()==true should imply isEmpty()==true which is not the case for box two. 2) GeoDataLatLonBox box; GeoDataLatLonBox otherBox(1,2,3,4); box = box.united(otherBox); In 2) I'd like to have box == otherBox, which would work if the default ctor was constructing a null box. On a related note, the latter case (uniting with a default constructed box) leads to a possible bug in the bound calculation. For example, uniting (showing west and east bounds in degrees now for each) [-180 180] with [8.41941 8.42429] results in [180 8.42429] Reproducible: Always
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