Skip to content

fixed -Wzero-as-null-pointer-constant Clang warnings#232

Merged
danmar merged 2 commits intocppcheck-opensource:masterfrom
firewave:nullptr
Feb 12, 2022
Merged

fixed -Wzero-as-null-pointer-constant Clang warnings#232
danmar merged 2 commits intocppcheck-opensource:masterfrom
firewave:nullptr

Conversation

@firewave
Copy link
Copy Markdown
Collaborator

@firewave firewave commented Jan 27, 2022

This adds nullptr as a define when in c++03 mode allowing the code to be compiled against c++11 code without raising such warnings. AppleClang always defines it - even in c++03 mode.

@firewave firewave marked this pull request as draft January 27, 2022 18:41
@danmar
Copy link
Copy Markdown
Collaborator

danmar commented Jan 27, 2022

I think it's ok to replace 0 with nullptr.

@firewave firewave marked this pull request as ready for review January 27, 2022 19:05
@firewave
Copy link
Copy Markdown
Collaborator Author

I took care of the "hanging" ubuntu-16.04 jobs in #233

@danmar danmar merged commit 479d222 into cppcheck-opensource:master Feb 12, 2022
@firewave firewave deleted the nullptr branch February 12, 2022 18:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants