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.

Sign up for GitHub

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

Jump to bottom

P1951 Default Arguments for pair's Forwarding Constructor #689

Closed
wg21bot opened this issue Dec 11, 2019 · 9 comments · Fixed by cplusplus/draft#4675
Closed

P1951 Default Arguments for pair's Forwarding Constructor #689

wg21bot opened this issue Dec 11, 2019 · 9 comments · Fixed by cplusplus/draft#4675
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
2021-telecon

Comments

@wg21bot
Copy link
Collaborator

wg21bot commented Dec 11, 2019

P1951R0 Default Arguments for pair's Forwarding Constructor (Logan R. Smith)
@wg21bot wg21bot added the LWG Library label Dec 11, 2019
@wg21bot wg21bot added this to the 2020-02 milestone Dec 11, 2019
@JeffGarland JeffGarland added the lwg-prereview Needs LWG initial review before full group label Mar 24, 2020
@ben-craig ben-craig added B2 - improvement Bucket 2 as described by P0592: bug fixes, performance improvements, integration fixes for/between e size - small paper size estimate labels Oct 17, 2020
@JeffGarland
Copy link
Member

Discussion between LEWG and LWG chairs results in this paper taking a trip thru LEWG before LWG looks at it. Implementability is a concern.

@JeffGarland JeffGarland added LEWG Library Evolution and removed lwg-prereview Needs LWG initial review before full group labels Nov 14, 2020
@brycelelbach brycelelbach added ready-for-library-evolution-meeting-review This paper needs to be discussed at a Library Evolution meeting IS Ship vehicle: IS and removed LWG Library labels Nov 16, 2020
@jensmaurer jensmaurer modified the milestones: 2020-02, 2021-telecon Dec 28, 2020
@brycelelbach brycelelbach changed the title P1951 Default Arguments for pair's Forwarding Constructor P1951 Default Arguments for pair's Forwarding Constructor Jan 26, 2021
@brycelelbach brycelelbach added ready-for-library-evolution-electronic-poll This paper needs to undergo a Library Evolution electronic poll needs-revision Paper needs changes before it can proceed C++23 Targeted at C++23 and removed ready-for-library-evolution-meeting-review This paper needs to be discussed at a Library Evolution meeting labels Jan 26, 2021
@ben-craig
Copy link
Collaborator

P1951R0: Default Arguments for pair's Forwarding Constructor

P1951R0: Default Arguments for pair's Forwarding Constructor

2021-01-25 Library Evolution Telecon Minutes

Chair: Ben Craig

Champion: Logan Smith

Minute Taker: Mark Hoemmen

SUMMARY:

This paper fixes a long standing usability issue with std::pair. Similar issues exist with std::tuple, but language facilities do not exist to default template parameter packs. This change has been prototyped and tested in libc++.

Please add the implementation and testing experience to the next paper. Rebase the paper onto the working draft.

POLL: Send the next revision of P1951 (P1951R1) (Default Arguments for pair's Forwarding Constructor) to electronic balloting to LWG for C++23, with priority bucket 2 (Improvement), pending wording help from Bryce Adelstein Lelbach and Ville Voutilainen

Strongly Favor Weakly Favor Neutral Weakly Against Strongly Against
18 9 0 0 0

Attendance: 30

# of Authors: 1

Author Position: SF

Outcome: Unanimous consent, please bring a new revision with wording updates and documented implementation experience

POLL: Put this paper directly in the Winter polling period.

Attendance: 29

Outcome: No objections to unanimous consent

OUTCOME:

Modify P1951R0 (Default Arguments for pair's Forwarding Constructor) by rebasing the wording on the working draft, and then send the revised paper to electronic balloting to be forwarded to LWG for C++23, classified as an improvement of an existing feature (P0592R4 bucket 2 item).

@JeffGarland
Copy link
Member

JeffGarland commented Feb 2, 2021 •
edited

Tentatively scheduled for review 2020-02-12 - new version on wiki.

https://wiki.edg.com/pub/Wg21telecons2021/LibraryWorkingGroup/D1951R1.html

@JeffGarland JeffGarland added LWG Library lwg-fullreview Paper is ready for lwg full group review and removed needs-revision Paper needs changes before it can proceed labels Feb 2, 2021
@JeffGarland
Copy link
Member

Review held 2020-01-12
https://wiki.edg.com/bin/view/Wg21telecons2021/D1951-20210212

poll: pending approval from LEWG electronic pool do we forward P1951r1 for inclusion into C++23?

F A N
12 0 1

@JeffGarland JeffGarland added lwg-wait-for-lewg-electronic-poll and removed lwg-fullreview Paper is ready for lwg full group review labels Feb 12, 2021
@wg21bot
Copy link
Collaborator Author

wg21bot commented Feb 22, 2021

P1951R1 Default Arguments for pair's Forwarding Constructor (Logan R. Smith)

@brycelelbach brycelelbach added ready-for-library-evolution-electronic-poll This paper needs to undergo a Library Evolution electronic poll scheduled-for-library-evolution-electronic-poll and removed ready-for-library-evolution-electronic-poll This paper needs to undergo a Library Evolution electronic poll labels Feb 22, 2021
@brycelelbach brycelelbach removed the ready-for-library-evolution-electronic-poll This paper needs to undergo a Library Evolution electronic poll label Feb 22, 2021
@brycelelbach
Copy link

brycelelbach commented Mar 13, 2021 •
edited

2021 Winter Library Evolution Polls

P2333: 2021 Winter Library Evolution Poll Outcomes

POLL 6: Modify P1951R0 (Default Arguments For pair's Forwarding Constructor) by rebasing the wording on the working draft, and then send the revised paper to LWG for C++23, classified as an improvement of an existing feature (P0592R4 bucket 2 item).

Strongly Favor Weakly Favor Neutral Weakly Against Strongly Against
14 9 2 0 0

Outcome: Strong consensus in favor.

Outcome

Modify P1951R0 (Default Arguments For pair's Forwarding Constructor) by rebasing the wording on the working draft, and then send the revised paper to LWG for C++23, classified as an improvement of an existing feature (P0592R4 bucket 2 item).

@brycelelbach
Copy link

@JeffGarland I marked this tentatively-ready-for-lwg, hopefully that is the right status.

@JeffGarland JeffGarland added tentatively-ready-for-plenary Reviewed between meetings; ready for a vote. and removed size - small paper size estimate tentatively-ready-for-lwg labels Mar 13, 2021
@JeffGarland
Copy link
Member

LWG is done per notes above -- moving to tentatively-ready-for-plenary.

@JeffGarland JeffGarland removed the B2 - improvement Bucket 2 as described by P0592: bug fixes, performance improvements, integration fixes for/between e label Mar 13, 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 Jun 7, 2021
@jensmaurer jensmaurer mentioned this issue Jun 9, 2021
Merged
@wg21bot
Copy link
Collaborator Author

wg21bot commented Oct 26, 2021

Adopted 2021-06.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
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
Milestone
2021-telecon
Development

Successfully merging a pull request may close this issue.

P1951R1 Default Arguments for pair's Forwarding Constructor cplusplus/draft
6 participants
@brycelelbach @ben-craig @cor3ntin @JeffGarland @jensmaurer @wg21bot