----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126397/#review89648 -----------------------------------------------------------
Looks like this boils down to multiple qApp->desktop()->size() calls, ie. displayWidth/displayHeight in kwinglobals.h, right? We could just "borrow" that code and kick the qApp dep then? - Thomas Lübking On Dec. 17, 2015, 9:21 a.m., Martin Gräßlin wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/126397/ > ----------------------------------------------------------- > > (Updated Dec. 17, 2015, 9:21 a.m.) > > > Review request for KDE Frameworks, kwin and Albert Astals Cid. > > > Bugs: 354811 > https://bugs.kde.org/show_bug.cgi?id=354811 > > > Repository: kwindowsystem > > > Description > ------- > > We observed that with Compiz as window manager various applications > crash if they use QGuiApplication instead of QApplication. The reason > for this is that on Compiz the mapViewport code paths are used and > that has a widgets dependency. > > This change should ensure that applications do at least not crash > in this condition. > > BUG: 354811 > > > Diffs > ----- > > src/platforms/xcb/kwindowsystem.cpp > 9d287043c24894ca3c29c439c7939b139da055e8 > > Diff: https://git.reviewboard.kde.org/r/126397/diff/ > > > Testing > ------- > > User in referenced bug report tested it, works. > > > Thanks, > > Martin Gräßlin > >
_______________________________________________ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel
