----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/105024/#review14270 -----------------------------------------------------------
This review has been submitted with commit f4d08af8d2b41002ff2cd88ca9678a0131667260 by Aurélien Gâteau to branch master. - Commit Hook On May 24, 2012, 11:53 a.m., Aurélien Gâteau wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/105024/ > ----------------------------------------------------------- > > (Updated May 24, 2012, 11:53 a.m.) > > > Review request for Plasma. > > > Description > ------- > > This commit changes the behavior of the "runners" property when no manager > has been created yet: > - Setting the property emits runnersChanged() (unless it is the same as the > old value) > - Reading the property after setting it returns the new value > > > Diffs > ----- > > plasma/declarativeimports/krunnermodel/runnermodel.cpp 2717ef8 > > Diff: http://git.reviewboard.kde.org/r/105024/diff/ > > > Testing > ------- > > Run attached test script. Without the patch, output is: > """ > runners before: > runners after: > """ > > With the patch one gets: > """ > runners before: > onRunnersChanged(): runners=services,shell > runners after: services,shell > """ > > > Thanks, > > Aurélien Gâteau > >
_______________________________________________ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel