graesslin added inline comments.
INLINE COMMENTS
> davidedmundson wrote in input.cpp:960
> if we're always going to do:
>
> if (actionResult.first) {
> return actionResult.second;
>
> why return two bools and not just "and" them here?
if actionResult.first is true we should not further process and return the
result of actionResult.second. If actionResult.second is false we don't know
anymore what the result of actionResult.first was if we and them.
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