> On Lug. 7, 2015, 12:02 p.m., Luca Beltrame wrote: > > I can't comment on the code itself, but from my small unscientific test, > > the CPU usage of the component seems gone. > > Luca Beltrame wrote: > If that's the case (can anyone else confirm), it should also fix bug > 346134.
Unfortunately, I tested on another system and bug 346134 is still present, likely caused by something else. - Luca ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/124280/#review82167 ----------------------------------------------------------- On Lug. 7, 2015, 2:16 p.m., David Edmundson wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/124280/ > ----------------------------------------------------------- > > (Updated Lug. 7, 2015, 2:16 p.m.) > > > Review request for Plasma. > > > Bugs: 346134 and 348385 > https://bugs.kde.org/show_bug.cgi?id=346134 > https://bugs.kde.org/show_bug.cgi?id=348385 > > > Repository: kdeclarative > > > Description > ------- > > Test compared rounded QSize with QSizeF, which will be almost always > returning true. > > CCBUG: 348385 > REVIEW: > > > Diffs > ----- > > src/qmlcontrols/kquickcontrolsaddons/plotter.cpp > 9a939c3626fc6321a39e898b88e5b69c14b4e1b7 > > Diff: https://git.reviewboard.kde.org/r/124280/diff/ > > > Testing > ------- > > > Thanks, > > David Edmundson > >
_______________________________________________ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel