[Marble-devel] Review Request 108687: Introduce GeoDataDocument::baseUrl

Thibaut Gridel tgridel at free.fr
Sun Feb 3 10:07:45 UTC 2013


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

Ship it!


Looks good and solves the relative url issue in an elegant way.
Are exotic cases from relative file names in FileLoader impacted?


src/lib/geodata/data/GeoDataDocument.h
<http://git.reviewboard.kde.org/r/108687/#comment20182>

    Can you add some comments so we understand its intent?



src/lib/geodata/data/GeoDataObject.cpp
<http://git.reviewboard.kde.org/r/108687/#comment20183>

    Maybe for clarity the variable can be renamed documentRoot and take either the baseUrl or the documentFile.absolutePath directly?


- Thibaut Gridel


On Jan. 31, 2013, 7:21 p.m., Dennis Nienhüser wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/108687/
> -----------------------------------------------------------
> 
> (Updated Jan. 31, 2013, 7:21 p.m.)
> 
> 
> Review request for Marble and Thibaut Gridel.
> 
> 
> Description
> -------
> 
> Adds GeoDataDocument::baseUrl used by the kml/kmz plugin and resolvePath(). This way both .kmz and .kml files correctly load relative resources.
> 
> 
> Diffs
> -----
> 
>   src/lib/geodata/data/GeoDataDocument.h 46c9339 
>   src/lib/geodata/data/GeoDataDocument.cpp 3537094 
>   src/lib/geodata/data/GeoDataDocument_p.h 41f4b15 
>   src/lib/geodata/data/GeoDataObject.cpp abc10f4 
>   src/plugins/runner/kml/KmlRunner.cpp ef07219 
> 
> Diff: http://git.reviewboard.kde.org/r/108687/diff/
> 
> 
> Testing
> -------
> 
> http://kml-samples.googlecode.com/svn-history/r212/trunk/kml/Region/GroundOverlay/usa-ca-sf.kmz
> - Download and open usa-ca-sf.kmz in Marble
> - Unzip usa-ca-sf.kmz and load its doc.kml in Marble
> 
> 
> Thanks,
> 
> Dennis Nienhüser
> 
>

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


More information about the Marble-devel mailing list