Skip to content

VSO_0000000_initialize_everything: Avoid compiler warnings - #5655

Merged
Stephan T. Lavavej (StephanTLavavej) merged 2 commits into
microsoft:mainfrom
YexuanXiao:initialize_everything
Aug 8, 2025
Merged

VSO_0000000_initialize_everything: Avoid compiler warnings#5655
Stephan T. Lavavej (StephanTLavavej) merged 2 commits into
microsoft:mainfrom
YexuanXiao:initialize_everything

Conversation

@YexuanXiao

Copy link
Copy Markdown
Contributor

Clang also generates the -Wnontrivial-memcall warning for this test, similar to #5654.

Comment thread tests/std/tests/VSO_0000000_initialize_everything/test.cpp Outdated
@StephanTLavavej Stephan T. Lavavej (StephanTLavavej) added the test Related to test code label Aug 2, 2025
@StephanTLavavej Stephan T. Lavavej (StephanTLavavej) changed the title Suppress Clang warnings for tests/VSO_0000000_initialize_everything VSO_0000000_initialize_everything: Avoid compiler warnings Aug 2, 2025
@StephanTLavavej Stephan T. Lavavej (StephanTLavavej) moved this from Initial Review to Ready To Merge in STL Code Reviews Aug 2, 2025
@StephanTLavavej Stephan T. Lavavej (StephanTLavavej) moved this from Ready To Merge to Merging in STL Code Reviews Aug 7, 2025
@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 f9b486e into microsoft:main Aug 8, 2025
39 checks passed
@github-project-automation github-project-automation Bot moved this from Merging to Done in STL Code Reviews Aug 8, 2025
@StephanTLavavej

Copy link
Copy Markdown
Member

⚠️ 😺 ⚠️

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

Labels

test Related to test code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants