anthonyfieroni added inline comments.

INLINE COMMENTS

> input.cpp:1734-1738
>          connect(conn, &LibInput::Connection::eventsRead, this,
>              [this] {
>                  m_libInput->processEvents();
>              }, Qt::QueuedConnection
>          );

Doesn't it to be in setup?

  void Connection::setup(QObject *to)
  {
      connect(this, &LibInput::Connection::eventsRead, to, [this]() { 
processEvents(); }, Qt::QueuedConnection);
      QMetaObject::invokeMethod(this, "doSetup", Qt::QueuedConnection);
  }

Caller should be

  conn->setup(this);

Otherwise it looks inconvenience, no?

REPOSITORY
  R108 KWin

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

To: fvogt, #plasma, graesslin, davidedmundson
Cc: anthonyfieroni, ngraham, kwin, #kwin, plasma-devel, bwowk, ZrenBot, 
progwolff, lesliezhai, ali-mohamed, hardening, jensreuterberg, abetts, sebas, 
apol, mart

Reply via email to