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

[Friedrich W. H. Kossebau]

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

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


Review request for Marble.


Changes
-------

Updated patch:
* removed unused method in LatLonParser
* changed m_dirCapExp to just match a word of max direction length, content will be checked later in more detail anyway 


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 (updated)
-----

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

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


Testing
-------

Patch adds 3 new unit tests which all pass.


Thanks,

Friedrich W. H. Kossebau

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