Skip to content

Update comments for approved LWG issues - #3238

Merged
Stephan T. Lavavej (StephanTLavavej) merged 1 commit into
microsoft:mainfrom
CaseyCarter:comment-cleanup
Nov 19, 2022
Merged

Stephan T. Lavavej (StephanTLavavej) merged 1 commit into
microsoft:mainfrom
CaseyCarter:comment-cleanup

Conversation

@CaseyCarter

@CaseyCarter Casey Carter (CaseyCarter) commented Nov 16, 2022

Copy link
Copy Markdown
Contributor

We no longer need comments for LWG issues that are now Standard. This includes comment cleanup for:

This also removes /* strengthened */ comments for functions that are now noexcept:

  • atomic_meow functions made noexcept by LWG-3745, and
  • reference_wrapper::operator() made conditionally noexcept by LWG-3764.

We no longer need comments for LWG issues that are now Standard. This includes comment cleanup for:
* LWG-3569,
* LWG-3766, and
* LWG-3788.

This also removes `/* strengthened */` comments for functions that are now `noexcept`:
* `atomic_meow` functions made `noexcept` LWG-3745, and
* `reference_wrapper::operator()` made conditionally `noexcept` by LWG-3764.
@CaseyCarter Casey Carter (CaseyCarter) added the documentation Related to documentation or comments label Nov 16, 2022
@CaseyCarter
Casey Carter (CaseyCarter) requested a review from a team as a code owner November 16, 2022 23:43
Comment thread stl/inc/atomic
@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 e28f956 into microsoft:main Nov 19, 2022
@StephanTLavavej

Copy link
Copy Markdown
Member

Thanks for aligning the codebase with the latest Working Draft! 😸 🎉 🐈

@CaseyCarter
Casey Carter (CaseyCarter) deleted the comment-cleanup branch November 19, 2022 02:26
@Tojfel

Tojfel commented Nov 29, 2022

Copy link
Copy Markdown

#@ Duplicate of #

@Tojfel

Tojfel commented Nov 29, 2022

Copy link
Copy Markdown

``// Use 'postCreateCommand' to run commands after the container is created.
"postCreateCommand": "dotnet restore",

@CaseyCarter

Copy link
Copy Markdown
Contributor Author

Tojfel are these intentional posts? I have no idea what you're trying to say.

@AraHaan

AraHaan commented Nov 29, 2022

Copy link
Copy Markdown

I think they accidentally posted those here when they intended to post them in another repository. I am also guilty of that sometimes too when tabs blend in from edge shortening tab lengths to just the icons.

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.

4 participants