----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/124375/#review82572 -----------------------------------------------------------
Surely all this patch will do is change it from infinitely recursing to 100% CPU busy looping in each event loop? I don't see how that's much better? - David Edmundson On July 16, 2015, 1:48 p.m., Daniel Vrátil wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/124375/ > ----------------------------------------------------------- > > (Updated July 16, 2015, 1:48 p.m.) > > > Review request for Plasma. > > > Repository: plasma-workspace > > > Description > ------- > > `PanelView::event()` generates new events for itself and dispatches them via > `QCoreApplication::sendEvent()`, which calls the handler directly. This can > lead to endless recursion. This patch changes the handler to use > `QCoreApplication::postEvent()` instead to enqueue the new event and dispatch > it from `QEventLoop` after the current `PanelView::event()` returns. > > Downstream bug report: https://bugzilla.redhat.com/show_bug.cgi?id=1226644 > > > Diffs > ----- > > shell/panelview.cpp b94673d > > Diff: https://git.reviewboard.kde.org/r/124375/diff/ > > > Testing > ------- > > > Thanks, > > Daniel Vrátil > >
_______________________________________________ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
