D11414: [xcb] Fix implementation of _NET_WM_FULLSCREEN_MONITORS

Friedrich W. H. Kossebau noreply at phabricator.kde.org
Sat Mar 24 00:57:14 UTC 2018


kossebau added inline comments.

INLINE COMMENTS

> netwm.cpp:2845-2847
> +        const uint32_t data[5] = {
> +            topology.top, topology.bottom, topology.left, topology.right, 1
> +        };

Seems clang (at least as of FreeBSD with -Wc++11-narrowing) does not like this narrowing from the int topology properties to the uint32_t array one:

  00:37:23 /usr/home/jenkins/workspace/Dependency Build Frameworks kf5-qt5 FreeBSDQt5.9/kwindowsystem/src/platforms/xcb/netwm.cpp:2846:13: error: non-constant-expression cannot be narrowed from type 'int' to 'uint32_t' (aka 'unsigned int') in initializer list [-Wc++11-narrowing]
  00:37:23             topology.top, topology.bottom, topology.left, topology.right, 1
  00:37:23             ^~~~~~~~~~~~

https://build.kde.org/view/CI%20Management/job/Dependency%20Build%20Frameworks%20kf5-qt5%20FreeBSDQt5.9/25/console

Should this be fixed with some casting?

REPOSITORY
  R278 KWindowSystem

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

To: graesslin, #frameworks, #kwin, #plasma, davidedmundson
Cc: kossebau, michaelh, ngraham
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20180324/3882981f/attachment-0001.html>


More information about the Kde-frameworks-devel mailing list