----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/118869/#review60764 -----------------------------------------------------------
Ship it! Code-wise the patch makes sense to me. I'm also running it since a while with some different plasmoids that stress it (clocks, system-monitor related plasmoids) and seems to work fine. One thing, the same patch should be pushed in plasma-framework as well. - Marco Martin On June 22, 2014, 11:34 a.m., Christoph Feck wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/118869/ > ----------------------------------------------------------- > > (Updated June 22, 2014, 11:34 a.m.) > > > Review request for Plasma and Aaron J. Seigo. > > > Bugs: 336551 > http://bugs.kde.org/show_bug.cgi?id=336551 > > > Repository: kdelibs > > > Description > ------- > > As described in bug 336551, plasma data engines use QTime to find out about > elapsed time. The problem is that QTime handles time zones, and therefore > reads /etc/localtime on each call. > > Instead, it should use QElapsedTimer. This fixes both the performance issue, > as well as the FIXME from the comment about not handled 24-h wraps and > timezone changes. > > There are probably more places where this can be changed. > > > Diffs > ----- > > plasma/datacontainer.cpp d19b1a5 > plasma/dataengine.cpp 9612574 > plasma/private/datacontainer_p.h a3e1f00 > plasma/private/dataengine_p.h 74a61e2 > > Diff: https://git.reviewboard.kde.org/r/118869/diff/ > > > Testing > ------- > > > Thanks, > > Christoph Feck > >
_______________________________________________ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
