-
-
Notifications
You must be signed in to change notification settings - Fork 32.2k
gh-132314: suppress maybe-uninitialized
compiler warning
#134207
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
base: main
Are you sure you want to change the base?
Conversation
maybe-uninitialized
compiler warning
I don't see uninitialized
|
I gathered that the error is because the compiler does not know if You can call the buildbot |
I clicked the Update button accidentally somehow. |
The pr proposed 2 different approaches, what do you think about this one? |
I do not think that your approach here can silence the maybe uninitialized warning on |
@chris-eibl The other PR seems to have been forgotten, I propose we just ignore as normal then. |
Yeah, after all, your approach here deals just with clang, which is the only compiler that issues the warning so far, so most probably no need to go with something like |
Is it right that this is attached to gh-132314? if so, please add that issue number. |
Ups, I was too fast: in the issue and in your referred log
gcc is raising this warning ... |
maybe-uninitialized
compiler warningmaybe-uninitialized
compiler warning
maybe-uninitialized
compiler warningmaybe-uninitialized
compiler warning
This comment was marked as resolved.
This comment was marked as resolved.
This comment was marked as resolved.
This comment was marked as resolved.
@chris-eibl We have to move it, it did not work, see log. |
(Bonus: 3 space indentation :-)
Found in this buildbot log.