Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Strike surviving mentions of "storage class" (outside of stable names and grammar) #3906

Merged
merged 1 commit into from Nov 7, 2023

Conversation

sdkrystian
Copy link
Contributor

Outside of the grammar term and first paragraph of [dcl.stc], "storage class" has no meaning in C++, and has no need to be defined. This replaces mentions of "storage class" with storage-class-specifier, "storage duration" and "linkage" where appropriate.

Copy link
Member

@zygoloid zygoloid left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please follow the rules at https://github.com/cplusplus/draft/wiki/Commit-message-format when naming your commits.

source/classes.tex Outdated Show resolved Hide resolved
source/declarations.tex Outdated Show resolved Hide resolved
source/templates.tex Outdated Show resolved Hide resolved
@zygoloid zygoloid added the changes requested Changes to the wording or approach have been requested and not yet applied. label Mar 25, 2020
@sdkrystian sdkrystian force-pushed the patch-45 branch 4 times, most recently from 91a7d5e to ae76dfb Compare March 26, 2020 00:46
source/classes.tex Outdated Show resolved Hide resolved
@jensmaurer jensmaurer removed the changes requested Changes to the wording or approach have been requested and not yet applied. label Apr 15, 2020
Copy link
Member

@jensmaurer jensmaurer left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good.

source/classes.tex Outdated Show resolved Hide resolved
@zygoloid zygoloid added the changes requested Changes to the wording or approach have been requested and not yet applied. label Oct 18, 2020
@jensmaurer jensmaurer added the needs rebase The pull request needs a git rebase to resolve merge conflicts. label Oct 29, 2020
@jensmaurer
Copy link
Member

This needs a rebase + force-push, plus a small fix.

@tkoeppe
Copy link
Contributor

tkoeppe commented Dec 14, 2020

@sdkrystian: Ping

@sdkrystian
Copy link
Contributor Author

@jensmaurer @zygoloid @tkoeppe Apologies for the inactivity, school has kept me very busy and unable to do much wg21 related work. At the end of the semester (little over a week) I'll be able to address everything that needs changes or rebasing.

@jensmaurer
Copy link
Member

@sdkrystian: Ping^2

@tkoeppe
Copy link
Contributor

tkoeppe commented Aug 18, 2022

@sdkrystian: Ping^3

@sdkrystian
Copy link
Contributor Author

@jensmaurer @tkoeppe Very sorry for disappearing for the last few years. Rebased, updated commit message, and applied suggested wording.

@tkoeppe
Copy link
Contributor

tkoeppe commented Nov 7, 2023

No need to apologize! And thanks for the contribution!

@tkoeppe tkoeppe merged commit dc6eed0 into cplusplus:main Nov 7, 2023
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
changes requested Changes to the wording or approach have been requested and not yet applied. needs rebase The pull request needs a git rebase to resolve merge conflicts.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants