[Marble-devel] Review Request: PositionTracking: name the placemarks and improve tracking representation

[Thibaut Gridel]

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

Review request for marble.


Summary
-------

Naming the Placemarks removes the "empty placemark" on top of placemark list.
Creating a new PolyLine each time the position provider status becomes "available" explicits the fact that a position change has happened without tracking points being updated.


Diffs
-----

  /trunk/KDE/kdeedu/marble/src/lib/gps/PositionTracking.h 1151760 
  /trunk/KDE/kdeedu/marble/src/lib/gps/PositionTracking.cpp 1151760 

Diff: http://reviewboard.kde.org/r/4725/diff


Testing
-------

Search for Current Position and Current Track in search line.
Click on those recenters on position or center of track.
Run with gpsfake, toggle position tracking source.


Thanks,

Thibaut

-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://mail.kde.org/pipermail/marble-devel/attachments/20100720/3ea56090/attachment.htm