Skip to content

implement P1951R1: Default Template Arguments For pair's Forwarding Constructor#2009

Merged
StephanTLavavej merged 10 commits into
microsoft:mainfrom
fsb4000:fix1976
Jul 30, 2021
Merged

implement P1951R1: Default Template Arguments For pair's Forwarding Constructor#2009
StephanTLavavej merged 10 commits into
microsoft:mainfrom
fsb4000:fix1976

Conversation

@fsb4000
Copy link
Copy Markdown
Contributor

@fsb4000 fsb4000 commented Jun 13, 2021

Fixes #1976

@fsb4000 fsb4000 requested a review from a team as a code owner June 13, 2021 14:56
@fsb4000 fsb4000 changed the title implement P1951R1 implement P1951R1: Default Template Arguments For pair's Forwarding Constructor Jun 13, 2021
Comment thread tests/std/tests/P1951R1_default_arguments_pair_forward_ctor/test.cpp Outdated
@StephanTLavavej StephanTLavavej added the cxx23 C++23 feature label Jun 14, 2021
Comment thread tests/std/tests/P1951R1_default_arguments_pair_forward_ctor/test.compile.pass.cpp Outdated
Comment thread tests/std/tests/P0067R5_charconv/test.cpp
@fsb4000 fsb4000 requested a review from CaseyCarter June 23, 2021 12:11
Comment thread tests/std/tests/P0067R5_charconv/test.cpp
@StephanTLavavej StephanTLavavej self-assigned this Jun 23, 2021
@StephanTLavavej

This comment has been minimized.

Comment thread stl/inc/yvals_core.h Outdated
@StephanTLavavej
Copy link
Copy Markdown
Member

FYI @CaseyCarter, I pushed changes after you approved, to guard this feature for C++23 (due to the breaking change impact and our usual conventions; we guard even innocuous stuff like shared_ptr::weak_type when doing so is easy). Note that C++23 implies _HAS_CONDITIONAL_EXPLICIT, so now only that branch is being changed.

@StephanTLavavej StephanTLavavej merged commit 7b59dff into microsoft:main Jul 30, 2021
@fsb4000 fsb4000 deleted the fix1976 branch July 30, 2021 02:58
@StephanTLavavej
Copy link
Copy Markdown
Member

Thanks for implementing this new feature in one of C++'s most popular types! 😺 🚀 ✔️

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cxx23 C++23 feature

Projects

None yet

Development

Successfully merging this pull request may close these issues.

P1951R1 Default Template Arguments For pair's Forwarding Constructor

5 participants