Stephan T. Lavavej

Results 357 comments of Stephan T. Lavavej

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

Thanks, this is awesome! :heart_eyes_cat: I pushed a series of fine-grained commits to follow our coding conventions in the test, but nothing serious (beyond a bug number typo). We merge...

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

Thanks for fixing this bug, and congratulations on your first microsoft/STL commit! :heart_eyes_cat: :tada: :magic_wand: This change is expected to ship in VS 2022 17.11 Preview 3.

This sounds like a reasonable analysis to us. We would want to consider PRs for this, with benchmarks, for individual algorithms at a time (not all at once please!). #813...

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

Thanks for noticing and fixing this inconsistency! :eagle: :eye: :magic_wand:

Thanks, this is great! :heart_eyes_cat: I pushed further changes to centralize the logic, please meow if you have concerns.

I'm drowning in PRs right now, so I think I'm going to have to clear out the backlog in multiple batches (in between investigating a non-STL bug that I can't...

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