Skip to content

Remove LWG-4472 comment from atomic_ref - #6358

Merged
Stephan T. Lavavej (StephanTLavavej) merged 1 commit into
microsoft:mainfrom
Ninetailedcatfox:cleanup-lwg4472-comment
Jul 17, 2026
Merged

Remove LWG-4472 comment from atomic_ref#6358
Stephan T. Lavavej (StephanTLavavej) merged 1 commit into
microsoft:mainfrom
Ninetailedcatfox:cleanup-lwg4472-comment

Conversation

@Ninetailedcatfox

@Ninetailedcatfox Jack Smith (Ninetailedcatfox) commented Jul 12, 2026

Copy link
Copy Markdown
Contributor

The fix for LWG-4472 (atomic_ref<const T> construction from temporaries) has already been implemented via the = delete on the rvalue-reference constructor. This removes the tracking comment now that the issue has been accepted.

Closes #6222

Copilot AI review requested due to automatic review settings July 12, 2026 07:07
@github-project-automation github-project-automation Bot moved this to Initial Review in STL Code Reviews Jul 12, 2026
@azure-pipelines

This comment was marked as resolved.

This comment was marked as resolved.

@StephanTLavavej Stephan T. Lavavej (StephanTLavavej) added the LWG Library Working Group issue label Jul 13, 2026
@StephanTLavavej

This comment was marked as resolved.

@azure-pipelines

This comment was marked as resolved.

@StephanTLavavej Stephan T. Lavavej (StephanTLavavej) added the documentation Related to documentation or comments label Jul 16, 2026
@StephanTLavavej Stephan T. Lavavej (StephanTLavavej) moved this from Initial Review to Ready To Merge in STL Code Reviews Jul 16, 2026
@StephanTLavavej Stephan T. Lavavej (StephanTLavavej) moved this from Ready To Merge to Merging in STL Code Reviews Jul 17, 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 a8a91c7 into microsoft:main Jul 17, 2026
49 checks passed
@github-project-automation github-project-automation Bot moved this from Merging to Done in STL Code Reviews Jul 17, 2026
@StephanTLavavej

Copy link
Copy Markdown
Member

Congrats on your first microsoft/STL commit! 🥇 😻 🎉

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

Labels

documentation Related to documentation or comments LWG Library Working Group issue

3 participants