[Marble-devel] Review Request 123202: Dynamic adding and removing of texture layers

[Adam Dabrowski]

> On April 1, 2015, 12:32 p.m., Dennis Nienhüser wrote:
> > src/lib/marble/layers/TextureLayer.cpp, line 258
> > <https://git.reviewboard.kde.org/r/123202/diff/1/?file=359672#file359672line258>
> >
> >     redundant.
> >     
> >     The whole block could also be replaced by `qDeleteAll( d->m_customTextures );`

good catch, I'll apply all these in next patch version.


- Adam


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/123202/#review78328
-----------------------------------------------------------


On April 1, 2015, 6:41 a.m., Adam Dabrowski wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/123202/
> -----------------------------------------------------------
> 
> (Updated April 1, 2015, 6:41 a.m.)
> 
> 
> Review request for Marble.
> 
> 
> Repository: marble
> 
> 
> Description
> -------
> 
> TextureLayer class was created based on parsing the .dgml content only. With this patch, MarbleMap API has been extended to allow a dynamic updates of texture layer (i.e. from the qml form-like interface, by pointing to a source directory and reading/inputting some parameters, by pointing to a geotiff and making tiles on the fly etc.). Theses custom textures are then added to static texture list and displayed.
> 
> 
> Diffs
> -----
> 
>   src/lib/marble/MarbleMap.h 260d46b 
>   src/lib/marble/MarbleMap.cpp 13becfc 
>   src/lib/marble/layers/TextureLayer.h fed6167 
>   src/lib/marble/layers/TextureLayer.cpp 9d8976f 
> 
> Diff: https://git.reviewboard.kde.org/r/123202/diff/
> 
> 
> Testing
> -------
> 
> Works well on my machine, tested by using for about 3-4 months now. Tested with 1, 2 and 3 dynamic texture layers (based on aerial photos).
> 
> 
> Thanks,
> 
> Adam Dabrowski
> 
>

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