D24244: Use lambdas for queued invocation of composition suspending

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
  Qt's metaobject is rather sensitive with scope resolution.
  
  Foo::Bar and Bar don't always match to a Qt metaobject, even if they
  refer to the same thing to a compiler. Here we register
  X11Compositor::SuspendReason but Q_ARG uses SuspendReason and they don't
  match. This leads to a runtime failure where the method isn't invoked.
  
  Rather than fixing metaobject usage, port the whole thing to lambdas
  which does better compile time checking and is generally nicer to read.
  
  BUG: 412353

TEST PLAN
  Ran xprop to block compositing. Compositing was blocked.
  Grepped source code for Q_ARG use

REPOSITORY
  R108 KWin

BRANCH
  master

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

AFFECTED FILES
  composite.cpp
  composite.h
  plugins/scenes/opengl/scene_opengl.cpp

To: davidedmundson, #kwin
Cc: kwin, LeGast00n, The-Feren-OS-Dev, sbergeron, jraleigh, fbampaloukas, GB_2, mkulinski, ragreen, jackyalcine, 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.