Skip to content

Remove comments for LWG-3210, LWG-4024, and LWG-4154 - #5360

Merged
Stephan T. Lavavej (StephanTLavavej) merged 2 commits into
microsoft:mainfrom
gabrielaugz:remove-lwg-comment-3210-4024-4154
Apr 10, 2025
Merged

Remove comments for LWG-3210, LWG-4024, and LWG-4154#5360
Stephan T. Lavavej (StephanTLavavej) merged 2 commits into
microsoft:mainfrom
gabrielaugz:remove-lwg-comment-3210-4024-4154

Conversation

@gabrielaugz

Copy link
Copy Markdown
Contributor

These comments referenced proposed resolutions for LWG-3210, LWG-4024, and LWG-4154, which have now been officially accepted as part of the Working Draft (November 2024 meeting). The implementations were already aligned with the standard, so the references are no longer necessary.

This PR simply removes those comments, with no behavioral changes.

Fixes #5127.

@gabrielaugz
Gabriel Augusto (gabrielaugz) force-pushed the remove-lwg-comment-3210-4024-4154 branch from 0f9cc6d to 8c860ff Compare March 25, 2025 18:36
@gabrielaugz

Copy link
Copy Markdown
Contributor Author

@microsoft-github-policy-service agree

@StephanTLavavej Stephan T. Lavavej (StephanTLavavej) added the documentation Related to documentation or comments label Mar 25, 2025
@StephanTLavavej

Copy link
Copy Markdown
Member

Thanks! Your PR was missing changes for LWG-4024 so I pushed a commit.

@StephanTLavavej Stephan T. Lavavej (StephanTLavavej) moved this from Initial Review to Ready To Merge in STL Code Reviews Mar 25, 2025
@StephanTLavavej

This comment was marked as resolved.

@azure-pipelines

This comment was marked as resolved.

@StephanTLavavej

Copy link
Copy Markdown
Member

I'm mirroring this to the MSVC-internal repo - please notify me if any further changes are pushed.

@StephanTLavavej
Stephan T. Lavavej (StephanTLavavej) merged commit d8d672f into microsoft:main Apr 10, 2025
@github-project-automation github-project-automation Bot moved this from Merging to Done in STL Code Reviews Apr 10, 2025
@StephanTLavavej

Stephan T. Lavavej (StephanTLavavej) commented Apr 10, 2025

Copy link
Copy Markdown
Member

Thanks for keeping the STL's comments up to date! 🎉 🚀 🐱

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Remove comments for LWG-3210, LWG-4024, and LWG-4154

2 participants