Review Request 122320: use xcb-screen count instead of qguiapplication.screens

Martin Gräßlin mgraesslin at kde.org
Mon Feb 2 07:14:35 GMT 2015


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/122320/#review75184
-----------------------------------------------------------



startkde/kcminit/CMakeLists.txt
<https://git.reviewboard.kde.org/r/122320/#comment51996>

    just find XCB as an optional dependency. You are not using XLib, so why bind it to whether XLib is found?



startkde/kcminit/CMakeLists.txt
<https://git.reviewboard.kde.org/r/122320/#comment51997>

    why this variable?



startkde/kcminit/main.cpp
<https://git.reviewboard.kde.org/r/122320/#comment51998>

    as you are still using an QGuiApplication I suggest to introduce a test for the platform name. The reason: on e.g. Wayland you don't want to check the xcb connection - which would succeed in the case of XWayland.


- Martin Gräßlin


On Feb. 1, 2015, 12:07 a.m., Nick Shaforostoff wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/122320/
> -----------------------------------------------------------
> 
> (Updated Feb. 1, 2015, 12:07 a.m.)
> 
> 
> Review request for kde-workspace, Martin Gräßlin and Thomas Lübking.
> 
> 
> Repository: plasma-workspace
> 
> 
> Description
> -------
> 
> this patch makes kcminit behave like in kde4: it uses proper xcb screen count which may be different from QGuiApplication::screens().count().
> 
> for example when i connext external monitor via vga to my laptop, xcb screen count is still '1', while QGuiApplication::screens().count() returns '2'.
> 
> switching from QGuiApplication to QCoreApplication still wasn't possible because modules like 'mouse' need gui initialized and would crash if kcminit uses QCoreApplication.
> 
> 
> Diffs
> -----
> 
>   startkde/kcminit/CMakeLists.txt b17951f 
>   startkde/kcminit/main.cpp 1008966 
> 
> Diff: https://git.reviewboard.kde.org/r/122320/diff/
> 
> 
> Testing
> -------
> 
> i have built kcminit on ubuntu vivid alpha 32-bit, replaced binaries and libraries in the system and successfuly could run kcminit_startup and reboot also went fine.
> 
> 
> Thanks,
> 
> Nick Shaforostoff
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20150202/81a5e551/attachment.htm>


More information about the kde-core-devel mailing list