> On April 4, 2013, 10:02 p.m., Matěj Laitl wrote: > > src/playback/PowerManager.cpp, lines 32-37 > > <http://git.reviewboard.kde.org/r/109846/diff/6/?file=131279#file131279line32> > > > > These seem to fit on one 90-char line
Also, the very first signal signature isn't normalized, please remove the space - Matěj ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/109846/#review30420 ----------------------------------------------------------- On April 4, 2013, 6:57 p.m., Anmol Ahuja wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/109846/ > ----------------------------------------------------------- > > (Updated April 4, 2013, 6:57 p.m.) > > > Review request for Amarok and Oliver Henshaw. > > > Description > ------- > > 1. Added "Suspend Options" to the PlaybackConfig Dialog, with two options: > Pause playback on suspend > Inhibit suspend if playing > 2. Created a PowerManager class to handle suspend behavior > > Fixes: > BR 259862 - Amarok does not inhibit suspend while playing > BR 222571 - Amarok pause on suspend > > > Diffs > ----- > > src/playback/PowerManager.h PRE-CREATION > src/configdialog/dialogs/PlaybackConfig.ui 3a79e43 > src/amarokconfig.kcfg fbe5497 > src/EngineController.cpp 52bfd90 > src/EngineController.h 5de4beb > src/CMakeLists.txt 990f313 > src/playback/PowerManager.cpp PRE-CREATION > > Diff: http://git.reviewboard.kde.org/r/109846/diff/ > > > Testing > ------- > > Works as expected > > > File Attachments > ---------------- > > PlaybackConfig Dialog > > http://git.reviewboard.kde.org/media/uploaded/files/2013/04/03/snapshot11.png > > > Thanks, > > Anmol Ahuja > >
_______________________________________________ Amarok-devel mailing list Amarok-devel@kde.org https://mail.kde.org/mailman/listinfo/amarok-devel