Conversation
This relatively unimportant pull request implements
the following changes:
- Fixed 3,427+ -Wzero-as-null-pointer-constant warnings when compiling
with Clang 22.1.8. This is primarily for code style consistency
(previous usage of 0 and NULL was seemingly random).
- Switched some struct init to C99 style incidentally to also
fix some -Wzero-as-null-pointer-constant warnings.
Progresses ksh93#777
- Fixed a few typing nits in conf.tab tests; also removed some fallback
test code for IRIX.
|
Nice; thanks. At long last we're rid of the 0/NULL annoyance. I'll need to verify that this builds on Solaris 11.4 with Solaris Studio cc; I seem to recall it doesn't like the use NULL (defined as |
|
Ah ha, I forgot I already took care of that three years ago (commit 0309d74) in features/standards, the results of which get included everywhere. ksh/src/lib/libast/features/standards Lines 143 to 151 in efc39a2 |
|
I'm getting a build error with this PR: |
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.
This relatively unimportant pull request implements the following changes:
-Wzero-as-null-pointer-constantwarnings when compiling with Clang 22.1.8. This is primarily for code style consistency (previous usage of 0 and NULL was seemingly random).-Wzero-as-null-pointer-constantwarnings. Progresses Migrate code to C99 #777