Silence two pointless warnings in Makefile.example#254
Open
xexyl wants to merge 2 commits intoioccc-src:masterfrom
Open
Silence two pointless warnings in Makefile.example#254xexyl wants to merge 2 commits intoioccc-src:masterfrom
xexyl wants to merge 2 commits intoioccc-src:masterfrom
Conversation
This is merely irony and satire and maybe a touch of sarcasm as well but importantly is right on theme. At the same time I fixed an error in bugs.html where the last entry there had text in the summary part of the file. Also I added a missing 'Jump to top' link in 2024/ferguson2 (also INABIAF). But still this is mostly for fun I just happened to notice the issues at the same time.
Not that there aren't many other pointless warnings but clang whines about '"missing" prototypes' and '"missing" variable declarations'. What it really means is that it wants you to put in static (or extern I guess but that's irrelevant in the IOCCC as you're not supposed to include your own header files to save bytes and for functions that is useless) despite the fact a variable can't be used before it's declared (hence not missing) and a function can't be called unless it's prototyped or the function comes first (hence not missing either). This is not only wasteful it is very annoying having to update the Makefile for every submission every year. These warnings are pointless outside the IOCCC in many cases but they're especially pointless in the IOCCC.
Contributor
Author
|
These to warnings are a bloody pain in my side .. and maybe something else :-) Should have done this ages ago because they're almost as bad as the '"unsafe" buffer usage'. UPDATEOkay actually they are as bad but just not as ridiculous. Or maybe they are but differently. |
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
1 participant
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.
Not that there aren't many other pointless warnings but clang whines
about '"missing" prototypes' and '"missing" variable declarations'. What
it really means is that it wants you to put in static (or extern I guess
but that's irrelevant in the IOCCC as you're not supposed to include
your own header files to save bytes and for functions that is useless)
despite the fact a variable can't be used before it's declared (hence
not missing) and a function can't be called unless it's prototyped or
the function comes first (hence not missing either). This is not only
wasteful it is very annoying having to update the Makefile for every
submission every year. These warnings are pointless outside the IOCCC in
many cases but they're especially pointless in the IOCCC.