----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/116641/#review52375 -----------------------------------------------------------
Ship it! I downloaded the patch and applied it. Works for me. New function names seem apt - Vedant Agarwala On March 6, 2014, 10:57 p.m., yash ladia wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/116641/ > ----------------------------------------------------------- > > (Updated March 6, 2014, 10:57 p.m.) > > > Review request for Amarok. > > > Bugs: 302299 > https://bugs.kde.org/show_bug.cgi?id=302299 > > > Repository: amarok > > > Description > ------- > > The bug was occuring since 'userAutoScrollOffset' and 'oldSliderPosition' > were being reset when track metadata was changed. This should not happen. > > I split the function: 'void _trackDataChanged( Meta::TrackPtr )' into 'void > _trackChanged( Meta::TrackPtr )' and 'void _trackMetadataChanged( > Meta::TrackPtr )' > > > Diffs > ----- > > src/context/applets/lyrics/LyricsApplet.h 20c71d8 > src/context/applets/lyrics/LyricsApplet.cpp 829a166 > > Diff: https://git.reviewboard.kde.org/r/116641/diff/ > > > Testing > ------- > > Builds and installs. > Expected results are obtained during testing. > > > Thanks, > > yash ladia > >
_______________________________________________ Amarok-devel mailing list Amarok-devel@kde.org https://mail.kde.org/mailman/listinfo/amarok-devel
