[Marble-devel] Review Request 110005: Deprecate VectorComposer

Bernhard Beschow bbeschow at cs.tu-berlin.de
Mon Sep 23 19:04:30 UTC 2013


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/110005/#review40604
-----------------------------------------------------------

Ship it!


I've commented out the painter state changes in {GeoLineString and GeoPolygon}GraphicsItem::paint() and benchmarked your patch again. I got roughly the same results: 61 fps w/ patch vs. ~60 w/o. Therefore, I consider the slight performance drop as identified and tolerable. Let's increase the performance in another patch.

Ship it!

- Bernhard Beschow


On April 13, 2013, 9:26 p.m., Thibaut Gridel wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/110005/
> -----------------------------------------------------------
> 
> (Updated April 13, 2013, 9:26 p.m.)
> 
> 
> Review request for Marble.
> 
> 
> Description
> -------
> 
> Uses the GeoData model and rendering instead of the legacy Vector code.
> Removed Classes are:
> - GeoPolygon
> - ScreenPolygon
> - VectorComposer
> - VectorMap
> - VectorMapBaseLayer and
> - VectorMapLayer
> 
> 
> Diffs
> -----
> 
>   data/maps/earth/bluemarble/bluemarble.dgml 4e40406 
>   data/maps/earth/citylights/citylights.dgml 8b72b60 
>   data/maps/earth/plain/plain.dgml 0e329c0 
>   data/maps/earth/precip-dec/precip-dec.dgml 77f38cf 
>   data/maps/earth/precip-july/precip-july.dgml f9e93f5 
>   data/maps/earth/schagen1689/schagen1689.dgml c68415c 
>   data/maps/earth/srtm/srtm.dgml d5b9810 
>   data/maps/earth/temp-dec/temp-dec.dgml 4d62d72 
>   data/maps/earth/temp-july/temp-july.dgml 394cb0e 
>   data/maps/sky/nightsky/nightsky.dgml f0d3777 
>   src/lib/CMakeLists.txt 534a7a9 
>   src/lib/GeoPolygon.h 37fb401 
>   src/lib/GeoPolygon.cpp ba9da66 
>   src/lib/MarbleMap.cpp 14ccd36 
>   src/lib/MarbleModel.cpp 5df138a 
>   src/lib/ScreenPolygon.h a1001d5 
>   src/lib/TextureColorizer.h 1fc2638 
>   src/lib/TextureColorizer.cpp f372830 
>   src/lib/VectorComposer.h b20db7c 
>   src/lib/VectorComposer.cpp 4d9600c 
>   src/lib/VectorMap.h 41a2e16 
>   src/lib/VectorMap.cpp cf3dcef 
>   src/lib/geodata/handlers/dgml/DgmlBrushTagHandler.cpp 50ce419 
>   src/lib/geodata/handlers/dgml/DgmlPenTagHandler.cpp 687ab61 
>   src/lib/geodata/handlers/dgml/DgmlSourceFileTagHandler.cpp f7c007f 
>   src/lib/geodata/handlers/dgml/DgmlVectorTagHandler.cpp b05ad8f 
>   src/lib/layers/CMakeLists.txt 4efe657 
>   src/lib/layers/TextureLayer.h 7c5f16c 
>   src/lib/layers/TextureLayer.cpp 1fe50d8 
>   src/lib/layers/VectorMapBaseLayer.h 4934cb9 
>   src/lib/layers/VectorMapBaseLayer.cpp 1bf1fbd 
>   src/lib/layers/VectorMapLayer.h 14e168d 
>   src/lib/layers/VectorMapLayer.cpp f05c88a 
>   tests/CMakeLists.txt d4cfec5 
>   tests/GeoPolygonTest.cpp 1f68f84 
> 
> Diff: http://git.reviewboard.kde.org/r/110005/diff/
> 
> 
> Testing
> -------
> 
> Existing dgml files have been ported to still use <vector> tag if doable, otherwise to directly use <geodata>
> All themes work, legend properties are functionnal still
> 
> 
> Thanks,
> 
> Thibaut Gridel
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/marble-devel/attachments/20130923/ef74263b/attachment.html>


More information about the Marble-devel mailing list