Skip to content

Cleanups for etc. - #3759

Merged
Stephan T. Lavavej (StephanTLavavej) merged 3 commits into
microsoft:mainfrom
achabense:_For_Mutex_Cleanups
Jun 15, 2023
Merged

Stephan T. Lavavej (StephanTLavavej) merged 3 commits into
microsoft:mainfrom
achabense:_For_Mutex_Cleanups

Conversation

@achabense

@achabense achabense commented Jun 10, 2023 •

Copy link
Copy Markdown
Contributor
  • Removes condition_variable::_Wait_until1(...const _timespec64*...), which is not called anywhere.
  • Fuses some ugly methods like timed_mutex::_Try_lock_until.

@achabense
achabense requested a review from a team as a code owner June 10, 2023 16:08
@StephanTLavavej Stephan T. Lavavej (StephanTLavavej) added the enhancement Something can be improved label Jun 10, 2023

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Perfect, thank you!

@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 641dd9b into microsoft:main Jun 15, 2023
@StephanTLavavej

Copy link
Copy Markdown
Member

Thanks for clearing out even more dead code! 💀 ☠️ 🪄

@achabense
achabense deleted the _For_Mutex_Cleanups branch June 15, 2023 11:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement Something can be improved

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants