----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/123411/#review79155 -----------------------------------------------------------
thanks for putting it on reviewboard. Overall it looks like we will need to do more work once we have support for it in Wayland. But to make it not crash it seems like a good approach at the moment. applets/pager/plugin/pager.h (line 144) <https://git.reviewboard.kde.org/r/123411/#comment54092> m_rows seems not to be initialized if m_isX11 is false applets/pager/plugin/pager.h (line 160) <https://git.reviewboard.kde.org/r/123411/#comment54091> make it: bool m_isX11 = false; and remove the #else #endif block in the initializer list - Martin Gräßlin On April 18, 2015, 4:07 a.m., Nerdopolis Turfwalker wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/123411/ > ----------------------------------------------------------- > > (Updated April 18, 2015, 4:07 a.m.) > > > Review request for Plasma. > > > Repository: plasma-desktop > > > Description > ------- > > Protect X11 calls in the Pager plasmoid to prevent plasmashell from crashing > in Wayland > > > Diffs > ----- > > applets/pager/plugin/pager.cpp 22c6ca2747f297dc3d37d02b46cb33196e64b1e0 > applets/pager/plugin/pager.h 78d461d2554e4885721da3d2eff82151c97f9305 > > Diff: https://git.reviewboard.kde.org/r/123411/diff/ > > > Testing > ------- > > Ran plasmoidviewer -a org.kde.plasma.pager in with QT_QPA_PLATFORM=wayland > got no crash from plasmoid viewer. > > Started an Xephyr session, set QT_QPA_PLATFORM=xcb and DISPLAY=:1, started > kwin_x11, and then plasmoidviewer -a org.kde.plasma.pager and had full > functionality. > > > Thanks, > > Nerdopolis Turfwalker > >
_______________________________________________ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
