[Marble-devel] Review Request 124421: Route preview was not working. Fixed!

[Dennis Nienhüser]

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


The code in question was added to fix https://bugs.kde.org/show_bug.cgi?id=342775
Can you check if you can reproduce the problem mentioned there?

- Dennis Nienhüser


On July 22, 2015, 6:08 p.m., Sanjiban Bairagya wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/124421/
> -----------------------------------------------------------
> 
> (Updated July 22, 2015, 6:08 p.m.)
> 
> 
> Review request for Marble, Dennis Nienhüser and Torsten Rahn.
> 
> 
> Repository: marble
> 
> 
> Description
> -------
> 
> There was an unnecessary check at the beginning of the function because of which the flow was never going down to the point where it should start playing the route as tour. Removed the, and the route preview takes place as required now.
> 
> 
> Diffs
> -----
> 
>   src/lib/marble/routing/RoutingWidget.cpp e3e6d9c 
> 
> Diff: https://git.reviewboard.kde.org/r/124421/diff/
> 
> 
> Testing
> -------
> 
> Tested. Works.
> 
> 
> Thanks,
> 
> Sanjiban Bairagya
> 
>

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