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

[expr.dynamic.cast] Remove redundant statements on casting away constness #1936

Merged
merged 1 commit into from Apr 2, 2018

Conversation

jensmaurer
Copy link
Member

and fix the null pointer provision.

Fixes #1934.

@jensmaurer jensmaurer changed the title [expr.dynamic.cast] Remove redundant statements on casting away consness [expr.dynamic.cast] Remove redundant statements on casting away constness Feb 19, 2018
@tkoeppe tkoeppe added the after-motions Pull request is to be applied after the pending edits from WG21 straw polls have been applied. label Mar 30, 2018
@zygoloid
Copy link
Member

zygoloid commented Apr 1, 2018

Per cwg discussion on #1453, the null pointer part of this is not editorial. Striking the redundant cv-qualification checks seems fine, though.

@jensmaurer
Copy link
Member Author

Removed edits for null pointer issue, but kept removing the redundant cv-qualification checks.

@jensmaurer jensmaurer requested a review from zygoloid April 1, 2018 15:49
@zygoloid zygoloid merged commit 1a9fd49 into cplusplus:master Apr 2, 2018
@jensmaurer jensmaurer deleted the b22 branch April 2, 2018 06:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
after-motions Pull request is to be applied after the pending edits from WG21 straw polls have been applied.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[expr.dynamic.cast] cv-qualification cleanups
3 participants