graesslin added inline comments.

INLINE COMMENTS

> kossebau wrote in netwm.cpp:2845-2847
> 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?

I guess we need to fix this

REPOSITORY
  R278 KWindowSystem

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

To: graesslin, #frameworks, #kwin, #plasma, davidedmundson
Cc: kossebau, michaelh, ngraham

Reply via email to