davidedmundson accepted this revision.
davidedmundson added a comment.
This revision is now accepted and ready to land.
ship it, but I have a question
INLINE COMMENTS
> input.cpp:960
> +
> +std::pair<bool, bool> performClientWheelAction(QWheelEvent *event,
> AbstractClient *c, MouseAction action = MouseAction::ModifierOnly)
> +{
if we're always going to do:
if (actionResult.first) {
return actionResult.second;
why return two bools and not just "and" them here?
REPOSITORY
R108 KWin
BRANCH
decoration-alt-button-5.11
REVISION DETAIL
https://phabricator.kde.org/D8758
To: graesslin, #kwin, #plasma, broulik, davidedmundson
Cc: davidedmundson, plasma-devel, kwin, bwowk, ZrenBot, progwolff, lesliezhai,
ali-mohamed, hardening, jensreuterberg, abetts, sebas, apol, mart
