Review Request 124230: [mousepad] Use QX11Info to get the Display
Martin Gräßlin
mgraesslin at kde.org
Thu Jul 2 12:55:24 UTC 2015
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/124230/
-----------------------------------------------------------
Review request for kdeconnect and Aleix Pol Gonzalez.
Repository: kdeconnect-kde
Description
-------
Don't open a second Display, we can use the one used by Qt.
Advantages are: we are on the correct Display, we don't open a Display
connection on e.g. Wayland (it would succeed but not work as a connection
to Xwayland would be opened) and we can drop the copy of fixx11h.
It's also a prerequisite to properly split the implementation to support
Wayland in a later change.
Diffs
-----
plugins/mousepad/CMakeLists.txt 823b54b8b19885e31f9af83c62d276cb316bedfa
plugins/mousepad/fixx11h.h 8d69e156d44b0c74e5f0540b2b400c4268430237
plugins/mousepad/mousepadplugin.h e8ce48028a253a3668871db2394400274b574c3d
plugins/mousepad/mousepadplugin.cpp f49aa7f31da7e208c75b30243138dbbd50f3101c
Diff: https://git.reviewboard.kde.org/r/124230/diff/
Testing
-------
compiles, extremely certain that it works without testing it.
Thanks,
Martin Gräßlin
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kdeconnect/attachments/20150702/2014ce1f/attachment.html>
More information about the KDEConnect
mailing list