Skip to content

LWG-3422: Issues of seed_seq's constructors - #2260

Merged
Stephan T. Lavavej (StephanTLavavej) merged 2 commits into
microsoft:mainfrom
MattStephanson:lwg3422
Oct 20, 2021
Merged

Stephan T. Lavavej (StephanTLavavej) merged 2 commits into
microsoft:mainfrom
MattStephanson:lwg3422

Conversation

@MattStephanson

Copy link
Copy Markdown
Contributor

Implements LWG-3422.

@CaseyCarter Casey Carter (CaseyCarter) added the LWG Library Working Group issue label Oct 6, 2021

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just one suggestion - since I can't review code without calling out something - in addition to your and Alex Guteniev (@AlexGuteniev)'s comments.

Comment thread stl/inc/random
@StephanTLavavej

Copy link
Copy Markdown
Member

Interestingly, <random> is mandated to include <initializer_list>, so I was unable to find any nitpicks with this PR. Perfect! 😹

@StephanTLavavej

Copy link
Copy Markdown
Member

I'm mirroring this to an MSVC-internal PR; please notify me if any further changes are pushed.

@StephanTLavavej
Stephan T. Lavavej (StephanTLavavej) merged commit ebd229f into microsoft:main Oct 20, 2021
@StephanTLavavej

Copy link
Copy Markdown
Member

Thanks for implementing this LWG issue resolution! 🌱 😸 ✔️

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

Labels

LWG Library Working Group issue

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants