Skip to content

Conversation

@CaseyCarter
Copy link
Contributor

@CaseyCarter CaseyCarter commented Dec 3, 2020

To unbreak RWC which uses the exact "return a lock from a function to allow the caller to do more work before unlocking" use case we were worried about.

Fixes VSO-1253916/AB#1253916.

To unbreak RWC which uses the exact "return a lock from a function to allow the caller to do more work before unlocking" use case we were worried about.
@CaseyCarter CaseyCarter added bug Something isn't working high priority Important! labels Dec 3, 2020
@CaseyCarter CaseyCarter requested a review from a team as a code owner December 3, 2020 20:22
@StephanTLavavej StephanTLavavej self-assigned this Dec 3, 2020
@StephanTLavavej StephanTLavavej changed the title Allow discarding unique_lock Allow discarding unique_lock and shared_lock Dec 3, 2020
@StephanTLavavej StephanTLavavej merged commit 60d9ed7 into microsoft:master Dec 3, 2020
@CaseyCarter CaseyCarter deleted the yesdiscard branch December 4, 2020 00:22
@StephanTLavavej
Copy link
Member

Thanks for fixing this Real World Code and ensuring that [[nodiscard]] remains valuable due to an extraordinarily low rate of false positives. 😻

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

Labels

bug Something isn't working high priority Important!

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants