-
Notifications
You must be signed in to change notification settings - Fork 6
Turn off some warnings for QoS for gcc13 #42
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do you have a link to the warning that this is preventing? I'll check if we can fix it in OpenDDS.
The new code is generating its own warning on compilers that don't have a -Wstringop-overflow
, so this may need to be conditional on the specific compiler/version:
https://github.com/OpenDDS/OpenDDW/actions/runs/15005619676/job/42163404066?pr=42#step:24:61
|
If we can't fix in OpenDDS, we can check for the warning so it doesn't cause warnings in other compilers. |
What was wrong with 65b076f? It looks correct to me and I didn't see any warnings on it. |
It didn't seem to work with gcc13 so it didn't fix the original problem sadly. The latest actually works. |
Is that issue with GCC 13 visible in the CI builds on ubuntu-24.04 (GCC 13 is the default version on Ubuntu 24.04)? If not, is there a configuration change we could make in the CI? |
Co-authored-by: Adam Mitz <[email protected]>
I had the issue when I built locally with gcc13. It looks like you are using gcc11 unless I am looking at the wrong thing. |
22.04 is GCC 11; 24.04 is GCC 13 https://github.com/OpenDDS/OpenDDW/actions/runs/15076340483/job/42384505992#step:24:27 |
Hum... I am confused then. It seems that you never got this warning with gcc13 in CI? |
I looked at the stackoverflow link again and discovered |
@iguessthislldo that makes sense but how come we are not seeing the warnings on the gcc13 build through CI. I see them when I build with gcc13 so its odd. |
I didn't see the warning in a local build with GCC 14 (I don't happen to have 13 installed on that particular VM anymore). As far as I can tell the warning from the compiler isn't meaningful and can be ignored. That's just a theory, but if we ran an OpenDDW-using program through valgrind or asan it might help to validate. |
Note: These warnings could be due to issues with OpenDDS but turn them off for now.