apol added a comment.

  +1

INLINE COMMENTS

> surface_interface.cpp:333
>      const bool childrenChanged = source->childrenChanged;
> +    const bool visibilityChanged = bool(source->buffer) ^ 
> bool(target->buffer);
>      bool sizeChanged = false;

Using != would probably be more readable and accurate (we're don't need it to 
be bitwise, we're assuming bool changes it to 00001 or 00000).

REPOSITORY
  R127 KWayland

BRANCH
  introduce-mapped-signal

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

To: zzag, #kwin, davidedmundson
Cc: apol, kde-frameworks-devel, LeGast00n, cblack, michaelh, ngraham, bruns

Reply via email to