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

P1147 Printing volatile Pointers #320

Closed
wg21bot opened this issue Jan 31, 2019 · 5 comments · Fixed by cplusplus/draft#5017
Closed

P1147 Printing volatile Pointers #320

wg21bot opened this issue Jan 31, 2019 · 5 comments · Fixed by cplusplus/draft#5017
Labels
C++23 Targeted at C++23 IS Ship vehicle: IS LWG Library plenary-approved Papers approved for inclusion in their target vehicle by plenary vote.
Milestone

Comments

@wg21bot
Copy link
Collaborator

wg21bot commented Jan 31, 2019

P1147R0 Printing volatile Pointers (Bryce Adelstein Lelbach)

@wg21bot wg21bot added this to the 2019-02 milestone Jan 31, 2019
@wg21bot wg21bot added the LEWG Library Evolution label Jan 31, 2019
@jensmaurer jensmaurer modified the milestones: 2019-02, 2019-07 Mar 20, 2019
@tituswinters
Copy link

Forward to LWG, we consider this a defect.
Unanimous consent

(We're happy for this to be in any ship vehicle that is considered reasonable.)

@tituswinters tituswinters added C++23 Targeted at C++23 LWG Library and removed LEWG Library Evolution labels Jul 19, 2019
@jensmaurer jensmaurer modified the milestones: 2019-07, 2019-11 Aug 23, 2019
@jensmaurer jensmaurer modified the milestones: 2019-11, 2020-06 Feb 18, 2020
@jensmaurer jensmaurer added IS Ship vehicle: IS and removed C++23 Targeted at C++23 labels Feb 18, 2020
@JeffGarland JeffGarland added the lwg-prereview Needs LWG initial review before full group label Mar 25, 2020
@ben-craig ben-craig added B2 - improvement Bucket 2 as described by P0592: bug fixes, performance improvements, integration fixes for/between e size - tiny paper size estimate, smaller than small labels Oct 17, 2020
@jensmaurer jensmaurer modified the milestones: 2020-telecon, 2021-telecon Dec 28, 2020
@JeffGarland JeffGarland added lwg-fullreview Paper is ready for lwg full group review and removed lwg-prereview Needs LWG initial review before full group labels Jul 23, 2021
@jwakely
Copy link
Member

jwakely commented Aug 20, 2021

LWG reviewed this today. The new member function is missing from the class synopsis, and the definition should use the injected-class-name, in line with cplusplus/draft#4831

We deemed a feature test macro unnecessary (you can just do the const_cast explicitly if you need to support implementations without the change).

Also, the drafting note at the start of the wording is unnecessary as there are no new sections using the character.

@JeffGarland
Copy link
Member

JeffGarland commented Aug 20, 2021

Adding to the above -- notes and votes from today

https://wiki.edg.com/bin/view/Wg21telecons2021/P1147-20210820

poll Move P1147r1 into the C++23 working paper?

F A N
6 0 0

@JeffGarland JeffGarland added C++23 Targeted at C++23 tentatively-ready-for-plenary Reviewed between meetings; ready for a vote. and removed lwg-fullreview Paper is ready for lwg full group review B2 - improvement Bucket 2 as described by P0592: bug fixes, performance improvements, integration fixes for/between e size - tiny paper size estimate, smaller than small labels Sep 18, 2021
@cor3ntin cor3ntin added plenary-approved Papers approved for inclusion in their target vehicle by plenary vote. and removed tentatively-ready-for-plenary Reviewed between meetings; ready for a vote. labels Oct 4, 2021
@wg21bot
Copy link
Collaborator Author

wg21bot commented Oct 26, 2021

P1147R1 Printing volatile Pointers (Bryce Adelstein Lelbach)

@wg21bot
Copy link
Collaborator Author

wg21bot commented Oct 26, 2021

Adopted 2021-10.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C++23 Targeted at C++23 IS Ship vehicle: IS LWG Library plenary-approved Papers approved for inclusion in their target vehicle by plenary vote.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

7 participants