You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
This isn't really negatively effecting me, but I noticed it in my build.
/home/jonesmz/osp-magnum-rmlui/3rdparty/corrade/src/Corrade/Utility/Arguments.h:173:30: warning: ‘Corrade::Utility::Arguments’ declared with greater visibility than the type of its field ‘Corrade::Utility::Arguments::_entries’ [-Wattributes]
173 | class CORRADE_UTILITY_EXPORT Arguments {
| ^~~~~~~~~
The text was updated successfully, but these errors were encountered:
Huh, I don't remember getting any similar warning locally. This is when you're including the header from your project, not when building Corrade itself, right?
GCC or Clang? Dynamic or static build? Any special flags you're setting in the superproject, like having -fvisibility=hidden but not -fvisibility-inlines-hidden? Does the error go away if you set neither or both?
Looks like it only happens when we include the file in one of our cpp files, and only with gcc and not clang (at least with the flags we're feeding them), and only on linux. Doesn't appear to happen on windows or macos. (Though we do get plenty of warnings building magnum on MSVC with -DCMAKE_SYSTEM_VERSION="10.0.19041.0", i just haven't gone through them to report issues yet, but you can look if you feel like it : https://github.com/TheOpenSpaceProgram/osp-magnum/runs/5082236880?check_suite_focus=true#step:10:51 )
In file included from /home/runner/work/osp-magnum/osp-magnum/src/test_application/main.cpp:48:
/home/runner/work/osp-magnum/osp-magnum/3rdparty/corrade/src/Corrade/Utility/Arguments.h:173:30: warning: ‘Corrade::Utility::Arguments’ declared with greater visibility than the type of its field ‘Corrade::Utility::Arguments::_entries’ [-Wattributes]
173 | class CORRADE_UTILITY_EXPORT Arguments {
| ^~~~~~~~~
This isn't really negatively effecting me, but I noticed it in my build.
The text was updated successfully, but these errors were encountered: