----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/111212/#review35031 -----------------------------------------------------------
Could you please also post the test case script you used? - Mark Kretschmann On June 24, 2013, 10:40 a.m., Anmol Ahuja wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/111212/ > ----------------------------------------------------------- > > (Updated June 24, 2013, 10:40 a.m.) > > > Review request for Amarok. > > > Description > ------- > > Added the ability to kill a long running script. > Refactored a bit. > > > Diffs > ----- > > src/ScriptManager.h afaa832 > src/ScriptManager.cpp b7f0a5c > src/amarokconfig.kcfg 6b4cefe > src/configdialog/dialogs/ScriptsConfig.ui b4b8d37 > src/scriptengine/AmarokScriptableServiceScript.cpp a312671 > > Diff: http://git.reviewboard.kde.org/r/111212/diff/ > > > Testing > ------- > > > Thanks, > > Anmol Ahuja > >
_______________________________________________ Amarok-devel mailing list Amarok-devel@kde.org https://mail.kde.org/mailman/listinfo/amarok-devel
