-
-
Notifications
You must be signed in to change notification settings - Fork 302
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
checks: Fix -Wsometimes-uninitialized compiler warnings #2165
Conversation
Can you please move the CodeQL errors to a separate PR? They are unrelated and likely we can merge them much sooner. |
63880e6
to
97894c3
Compare
I merged this with main which resolved conflicts, but it needs some additional clean up because now it would revert some changes which happened on main in the same files. Given that this is a draft without any actual code changes, I'm leaving it as is for now. |
80bfbe0
to
d034c1b
Compare
Finalised the rebase, to be able to continue with clean sheets. |
Opening this for review. I left the comments temporarily for context, I will remove them before merge. |
This is now ready, I'll proceed to merge soon if there are no objections. |
As reported in #2156.
Initially this is a draft, I commented in code to highlight the sometimes uninitialised variables.
We need to decide how to address them.
Affects:
Modules
GRASS Library parts