> On Aug. 3, 2016, 2:02 nachm., Kai Uwe Broulik wrote: > > kwindowsystem master fails to build: > > > > kwindowsystem/src/platforms/xcb/kwindowsystem.cpp:441:48: error: ‘class > > NETEventFilter’ has no member named ‘compositingEnabled’ > > const bool wasCompositing = s_d ? s_d->compositingEnabled : false; > > Martin Gräßlin wrote: > Please drop the patch https://git.reviewboard.kde.org/r/128566/
Ah, yeah, still had that one, sorry for the noise, works. - Kai Uwe ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/128576/#review98051 ----------------------------------------------------------- On Aug. 3, 2016, 9:34 vorm., Martin Gräßlin wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/128576/ > ----------------------------------------------------------- > > (Updated Aug. 3, 2016, 9:34 vorm.) > > > Review request for KDE Frameworks and Plasma. > > > Bugs: 362531 > https://bugs.kde.org/show_bug.cgi?id=362531 > > > Repository: kwindowsystem > > > Description > ------- > > The Xcb implementation of KWindowSystem has two operations modes and when > switching between the two it recreates the NETEventFilter. > > This could result in the compositingChanged signal never to be emitted if: > 1) NETEventFilter gets created before compositor is started > 2) NETEventFilter gets recreated after compositor is started but before > the old filter had a chance to process the XFixes event > > This was the cause for e.g. plasmashell not properly detecting that a > Compositor is running on X11. > > This change ensures that the signal is emitted if the compositing state > differs after the recreation. Also a test case is added which simulates > the condition. > > BUG: 362531 > > > Diffs > ----- > > autotests/CMakeLists.txt 690c41bfe2ca981d206c590c3fbd28416d4ae401 > autotests/compositingenabled_test.cpp PRE-CREATION > src/platforms/xcb/kwindowsystem.cpp > fbff381b61116c6eef47d5c7fa1fc5730857328c > > Diff: https://git.reviewboard.kde.org/r/128576/diff/ > > > Testing > ------- > > > Thanks, > > Martin Gräßlin > >
_______________________________________________ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
