Skip to content

Implement P3612R1 Harmonize Proxy-Reference Operations - #5848

Merged
Stephan T. Lavavej (StephanTLavavej) merged 19 commits into
microsoft:mainfrom
vmichal:P3612R1
Mar 4, 2026
Merged

Implement P3612R1 Harmonize Proxy-Reference Operations#5848
Stephan T. Lavavej (StephanTLavavej) merged 19 commits into
microsoft:mainfrom
vmichal:P3612R1

Conversation

@vmichal

@vmichal Vojtěch Michal (vmichal) commented Nov 12, 2025

Copy link
Copy Markdown
Contributor

Add friend swap functions to std::bitset::reference and std::vector::reference, add new assignment operators.
Add deprecation warning to std::vector::swap(reference, reference).

Resolves #5842

@vmichal

This comment was marked as resolved.

@vmichal

This comment was marked as resolved.

@StephanTLavavej

This comment was marked as resolved.

@StephanTLavavej

This comment was marked as resolved.

@StephanTLavavej Stephan T. Lavavej (StephanTLavavej) moved this from Initial Review to Work In Progress in STL Code Reviews Nov 12, 2025
@vmichal

This comment was marked as resolved.

@vmichal

This comment was marked as resolved.

@StephanTLavavej

This comment was marked as resolved.

@StephanTLavavej

This comment was marked as resolved.

@vmichal

This comment was marked as resolved.

@StephanTLavavej Stephan T. Lavavej (StephanTLavavej) moved this from Work In Progress to Initial Review in STL Code Reviews Nov 12, 2025
Comment thread stl/inc/bitset Outdated
Comment thread tests/std/tests/Dev10_816787_swap_vector_bool_elements/test.cpp
Comment thread stl/inc/yvals_core.h Outdated
Comment thread stl/inc/yvals_core.h Outdated
Comment thread tests/std/tests/Dev10_816787_swap_vector_bool_elements/test.cpp Outdated
Comment thread tests/std/tests/Dev10_816787_swap_vector_bool_elements/test.cpp Outdated
Comment thread tests/std/tests/Dev10_816787_swap_vector_bool_elements/test.cpp Outdated
Comment thread tests/std/tests/Dev10_816787_swap_vector_bool_elements/test.cpp Outdated
Comment thread tests/std/tests/Dev10_816787_swap_vector_bool_elements/test.cpp Outdated
Comment thread tests/std/tests/Dev10_816787_swap_vector_bool_elements/test.cpp Outdated
Comment thread tests/std/tests/Dev10_816787_swap_vector_bool_elements/test.cpp
Comment thread tests/std/tests/Dev10_816787_swap_vector_bool_elements/test.cpp Outdated
Comment thread tests/std/tests/Dev10_816787_swap_vector_bool_elements/test.cpp
@StephanTLavavej

Copy link
Copy Markdown
Member

Thanks! 😻 My apologies for how long it took to get around to reviewing this.

@StephanTLavavej Stephan T. Lavavej (StephanTLavavej) moved this from Initial Review to Ready To Merge in STL Code Reviews Mar 2, 2026
@StephanTLavavej Stephan T. Lavavej (StephanTLavavej) moved this from Ready To Merge to Merging in STL Code Reviews Mar 3, 2026
@StephanTLavavej

Copy link
Copy Markdown
Member

I'm mirroring this to the MSVC-internal repo. Please notify me if any further changes are pushed, otherwise no action is required.

@StephanTLavavej
Stephan T. Lavavej (StephanTLavavej) merged commit 7b05438 into microsoft:main Mar 4, 2026
49 checks passed
@github-project-automation github-project-automation Bot moved this from Merging to Done in STL Code Reviews Mar 4, 2026
@StephanTLavavej

Copy link
Copy Markdown
Member

Thanks for implementing this C++26 feature and deprecating the STL's most bizarre swap! 🎉 😸 🐈

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

Labels

cxx26 C++26 feature

4 participants