----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/119448/#review63077 -----------------------------------------------------------
src/lib/marble/routing/RoutingWidget.cpp <https://git.reviewboard.kde.org/r/119448/#comment43808> Still a memory leak for both the tour and its playback. Prepend each creation with a delete d->m_playback; and respectively delete d->m_tour; to delete any previous instance. Also delete both in the destructor. - Dennis Nienhüser On July 24, 2014, 2:40 p.m., Sanjiban Bairagya wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/119448/ > ----------------------------------------------------------- > > (Updated July 24, 2014, 2:40 p.m.) > > > Review request for Marble. > > > Repository: marble > > > Description > ------- > > This patch helps to view a route as a tour, by clicking on a toggled > Play/Pause button in the RoutingWidget when a route has been loaded > > > Diffs > ----- > > src/lib/marble/routing/RoutingWidget.cpp 7abee97 > src/lib/marble/TourPlayback.cpp 6682322 > src/lib/marble/routing/RoutingWidget.h b0e6b6d > > Diff: https://git.reviewboard.kde.org/r/119448/diff/ > > > Testing > ------- > > > Thanks, > > Sanjiban Bairagya > >
_______________________________________________ Marble-devel mailing list [email protected] https://mail.kde.org/mailman/listinfo/marble-devel
