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