[Marble-devel] Review Request: Fix GeoDataCoordinates::fromString(...)

Friedrich W. H. Kossebau kossebau at kde.org
Mon Oct 31 09:41:32 UTC 2011


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

(Updated Oct. 31, 2011, 9:41 a.m.)


Review request for Marble.


Description
-------

Fixes GeoDataCoordinates::fromString(...) by a rewrite of parsing code.
Turned parsing code into a parser class, named LatLonParser.

Supports parsing of D, DM, and DMS:
* last unit can be in floating point precision
* directions can be in local language, fallback to english if not matching
* accepts plain ASCII chars ' " as well as chars ? (PRIME) and ? (DOUBLE PRIME).


Diffs
-----

  src/lib/geodata/data/GeoDataCoordinates.cpp a0a2436 
  tests/unittest_geodatacoordinates.cpp 10f2d94 

Diff: http://git.reviewboard.kde.org/r/103000/diff/diff


Testing (updated)
-------

Patch adds 3 new unit tests which all pass.
Also entered coordinates in lots of different formats to my Plasma Marble Runner, worked fine.


Thanks,

Friedrich W. H. Kossebau

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


More information about the Marble-devel mailing list