Skip to content

<memory>: add _EXPORT_STD for start_lifetime_as and start_lifetime_as_array#6254

Merged
StephanTLavavej merged 1 commit intomicrosoft:mainfrom
Yuria-Shikibe:main
Apr 17, 2026
Merged

<memory>: add _EXPORT_STD for start_lifetime_as and start_lifetime_as_array#6254
StephanTLavavej merged 1 commit intomicrosoft:mainfrom
Yuria-Shikibe:main

Conversation

@Yuria-Shikibe
Copy link
Copy Markdown
Contributor

@Yuria-Shikibe Yuria-Shikibe commented Apr 15, 2026

I'm not sure if _EXPORT_STD is intentionally removed from start_lifetime_as and start_lifetime_as_array because without export msvc still successfully compiled code using import std; with start_lifetime_as and start_lifetime_as_array.

Anyway Resharper detected that they are not exported and leave red color on them, so I did this.

@Yuria-Shikibe Yuria-Shikibe requested a review from a team as a code owner April 15, 2026 03:10
@github-project-automation github-project-automation bot moved this to Initial Review in STL Code Reviews Apr 15, 2026
@Yuria-Shikibe
Copy link
Copy Markdown
Contributor Author

@microsoft-github-policy-service agree

@StephanTLavavej StephanTLavavej added the bug Something isn't working label Apr 15, 2026
@StephanTLavavej
Copy link
Copy Markdown
Member

Jeez, how did the author of #3108 miss this in #6170? What a joker.

Thanks for noticing and fixing this bug! 😻

@StephanTLavavej StephanTLavavej moved this from Initial Review to Ready To Merge in STL Code Reviews Apr 15, 2026
@StephanTLavavej StephanTLavavej changed the title <memory>: add _EXPORT_STD marco for start_lifetime_as and start_lifetime_as_array <memory>: add _EXPORT_STD for start_lifetime_as and start_lifetime_as_array Apr 15, 2026
@StephanTLavavej StephanTLavavej moved this from Ready To Merge to Merging in STL Code Reviews Apr 15, 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 StephanTLavavej merged commit 129a277 into microsoft:main Apr 17, 2026
49 checks passed
@github-project-automation github-project-automation bot moved this from Merging to Done in STL Code Reviews Apr 17, 2026
@StephanTLavavej
Copy link
Copy Markdown
Member

Thanks again for fixing my bug and congratulations 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

bug Something isn't working

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

2 participants