D23610: Fix orientation sensor DBus

David Edmundson <[email protected]>
Newsgroups gmane.comp.kde.devel.kwin
Message-ID <[email protected]>
davidedmundson created this revision.
davidedmundson added a reviewer: KWin.
Herald added a project: KWin.
Herald added a subscriber: kwin.
davidedmundson requested review of this revision.

REVISION SUMMARY
  Firstly, it was completely broken, no-one called registerObject.
  
  Secondly deleting the adaptor doesn't really make sense, you'd still
  leave the object valid, only have it broken. Docs of
  QDBusAbstractAdaptor do say not to ever delete it manually.
  
  Thirdly we don't need Q_CLASSINFO setting the DBus interface on the
  exported item when we use an adaptor.

TEST PLAN
  Manually added some setEnabled/disabled
  Could now see the path

REPOSITORY
  R108 KWin

BRANCH
  master

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

AFFECTED FILES
  orientation_sensor.cpp
  orientation_sensor.h

To: davidedmundson, #kwin
Cc: kwin, LeGast00n, The-Feren-OS-Dev, sbergeron, jraleigh, fbampaloukas, GB_2, mkulinski, ragreen, jackyalcine, Pitel, iodelay, crozbo, bwowk, ZrenBot, ngraham, alexeymin, himcesjf, lesliezhai, ali-mohamed, hardening, romangg, 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.