Skip to content

More precise check message for binary_semaphore - #2293

Merged
Stephan T. Lavavej (StephanTLavavej) merged 3 commits into
microsoft:mainfrom
AlexGuteniev:semahpore_dont_lie
Nov 3, 2021
Merged

Stephan T. Lavavej (StephanTLavavej) merged 3 commits into
microsoft:mainfrom
AlexGuteniev:semahpore_dont_lie

Conversation

@AlexGuteniev

Copy link
Copy Markdown
Contributor

Unlike counting_semaphore, precondition violations can't beat it down.
Only more aggressive sorts of UB can.

Unlike `counting_semaphore`, precondition violations can't beat it down.
Only more aggressive sorts of UB can.
@AlexGuteniev
Alex Guteniev (AlexGuteniev) requested a review from a team as a code owner October 22, 2021 13:36
Comment thread stl/inc/semaphore Outdated
Comment thread stl/inc/semaphore Outdated
Comment thread stl/inc/semaphore Outdated
Comment thread stl/inc/semaphore Outdated
@CaseyCarter Casey Carter (CaseyCarter) added the enhancement Something can be improved label Oct 22, 2021
"Corruption" is a mass noun
@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 7baa750 into microsoft:main Nov 3, 2021
@StephanTLavavej

Copy link
Copy Markdown
Member

Thanks for improving these diagnostics! ✔️ 🎉 😺

@AlexGuteniev
Alex Guteniev (AlexGuteniev) deleted the semahpore_dont_lie branch November 3, 2021 05:34
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.

4 participants