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

P2351: add [[nodiscard]] to cast-like library wrappers [WIP] #4579

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

hanickadot
Copy link
Contributor

bit_cast, to_integer, forward, move, move_if_noexcept, as_const, to_underlying, and identity::operator()

…except, as_const, to_underlying, and identity::operator()
@hanickadot hanickadot changed the title P2351: add [[nodiscard]] to: cast-like library wrappers [WIP] P2351: add [[nodiscard]] to cast-like library wrappers [WIP] Apr 5, 2021
@jensmaurer
Copy link
Member

Which part is "WIP" here?

@jensmaurer jensmaurer added the decision-required A decision of the editorial group (or the Project Editor) is required. label Apr 6, 2021
@hanickadot
Copy link
Contributor Author

Paper is WIP. LEWG is discussing if the paper is worth of spending time on it. My idea is to have an open pull request as part of the paper. If that's not what you want here close it.

@jensmaurer
Copy link
Member

@tkoeppe, please state whether you consider the change here editorial.
(My view: There is no general or specific guidance from LWG / LEWG for this matter, so this doesn't feel editorial.)

@hanickadot
Copy link
Contributor Author

No, I don't think this is editorial (as much I would like to). I want guidance from LEWG/LWG too. But I also like the idea that the paper is accompanied with an open pull request to the draft. To make it easier for editors.

@jensmaurer jensmaurer added lwg Issue must be reviewed by LWG. not-editorial Issue is not deemed editorial; the editorial issue is kept open for tracking. and removed decision-required A decision of the editorial group (or the Project Editor) is required. labels Apr 6, 2021
@wg21bot wg21bot added the needs rebase The pull request needs a git rebase to resolve merge conflicts. label Oct 24, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
lwg Issue must be reviewed by LWG. needs rebase The pull request needs a git rebase to resolve merge conflicts. not-editorial Issue is not deemed editorial; the editorial issue is kept open for tracking.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants