anthonyfieroni added inline comments.

INLINE COMMENTS

> cursorwrapper.cpp:29-30
> +{
> +    connect(Cursor::self(), &Cursor::posChanged, this, 
> &CursorWrapper::positionChanged);
> +    Cursor::self()->startCursorTracking();
> +}

https://phabricator.kde.org/source/kwin/browse/master/cursor.cpp;f50fbde1a9edae2eb9dd2947e457687dc94b41a7$207
startCursorTracking is written as recursive counter i.e. if external script 
enable mouse tracking - signal still emits. I'm not sure this is desired effect.

REPOSITORY
  R108 KWin

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

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

Reply via email to