Fix up some warnings/errors#792
Merged
mvandervoord merged 7 commits intoThrowTheSwitch:masterfrom Jul 9, 2025
Merged
Conversation
Member
|
Hi. Thanks for the collection of fixes! We really appreciate the help! This project supports more compilers than just clang and gcc, so we need to be a bit careful about some of the changes. If you don't mind making a few tweaks to better support those other compilers? For example:
Thanks so much! |
"empty expression statement has no effect; remove unnecessary ';'" These macros were not properly wrapped
These are reserved by the standard
This one was a bit tough, but I think this works fine.
Member
|
Thanks for the help, @RossSmyth ! |
Contributor
Author
Contributor
Author
|
Thanks! |
Member
|
:) Thank YOU! I'm not a meson user myself... so I intended to solicit a 2nd opinion on some of those... I'm sure I'll hear about it. ;) |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Close #791
My process:
Not all were successes. Some of the files are laid out in ways I don't really understand.
It is arranged in nice atomic commits, but can be squashed.
Tested with Clang and with GCC.