Skip to content

Update C++ standard list of --std option in manpage#5506

Merged
firewave merged 1 commit intocppcheck-opensource:mainfrom
syohex:std-list-in-man
Oct 4, 2023
Merged

Update C++ standard list of --std option in manpage#5506
firewave merged 1 commit intocppcheck-opensource:mainfrom
syohex:std-list-in-man

Conversation

@syohex
Copy link
Copy Markdown
Contributor

@syohex syohex commented Oct 3, 2023

The manpage only lists up to C++11 and it says C++11 is default. It is out-of-dated.

@firewave
Copy link
Copy Markdown
Collaborator

firewave commented Oct 4, 2023

This is probably not the one things which is out-of-sync. I filed https://trac.cppcheck.net/ticket/12035 about finding an approach so we only need to modify a single location.

@chrchr-github
Copy link
Copy Markdown
Collaborator

chrchr-github commented Oct 4, 2023

We even (pretend aspire to) support C++23 (which also is the default) in our code.

@firewave
Copy link
Copy Markdown
Collaborator

firewave commented Oct 4, 2023

We even (pretend aspire to) support C++23 (which also is the default) in our code.

Well, there is preliminary support in terms of keywords and the standard define. It is mostly important for the -std= compiler option we consume from projects. That is done via simplecpp though. That enables certain code for the analysis even if we do not support all the features.

That will actually become somewhat obsolete when we can deduce that directly from the compiler - but we are already well off the reservation, so let's get back on topic.

@syohex What name should we credit you with for your contribution?

@firewave firewave merged commit 354ea60 into cppcheck-opensource:main Oct 4, 2023
@syohex
Copy link
Copy Markdown
Contributor Author

syohex commented Oct 4, 2023

@syohex What name should we credit you with for your contribution?

Shohei Yoshida, please.

@syohex syohex deleted the std-list-in-man branch October 4, 2023 23:07
@firewave
Copy link
Copy Markdown
Collaborator

firewave commented Oct 5, 2023

Shohei Yoshida, please.

@danmar

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.

3 participants