D17472: Fix dbus export of property Device::supportedButtons

Martin Flöser <[email protected]>
Newsgroups gmane.comp.kde.devel.kwin
Message-ID <[email protected]>
graesslin created this revision.
graesslin added a reviewer: KWin.
Herald added a project: KWin.
Herald added a subscriber: kwin.
graesslin requested review of this revision.

REVISION SUMMARY
  When launching the touchpad kcm we see the following warning:
  
    Error on d-bus read of "supportedButtons"
  
  And the Gui shows "Error while loading values. See logs for more
  information. Please restart this configuration module".
  
  By extending the unit test for reading back the properties through DBus
  I noticed that it's not trivially possible to export QFlags to DBus. As
  the QFlags can be implicitly casted to int, the property is changed to
  an int value.

TEST PLAN
  Test case adjusted, did not pass prior to change, passes now.

REPOSITORY
  R108 KWin

BRANCH
  libinput-device-supported-buttons

REVISION DETAIL
  https://phabricator.kde.org/D17472

AFFECTED FILES
  autotests/libinput/device_test.cpp
  libinput/device.h

To: graesslin, #kwin
Cc: kwin, mkulinski, ragreen, jackyalcine, Pitel, iodelay, bwowk, ZrenBot, lesliezhai, ali-mohamed, hardening, jensreuterberg, abetts, sebas, apol, mart
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.