New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.
Starred by 5 users
Status: Archived
Owner: ----
Closed: Sep 19
NextAction: ----
OS: ----
Pri: 3
Type: Bug

Sign in to add a comment
CriticalSectionWrapper should have an AssertAcquired() method
Reported by, Sep 9 2013 Back to list
Came up in CR:
If CriticalSectionWrapper::AssertAcquired() existed then code could be better self-documenting.
CSW::AA() should assert(false) if the lock is not held and be a no-op in NDEBUG mode.
Sample impl in Chromium:
Comment 1 by, Dec 17 2014
Labels: Area-PeerConnection
Project Member Comment 2 by, Jan 6 2015
Labels: EngTriaged
Hey, pbos, who owns SystemWrappers?
Project Member Comment 3 by, Jan 7 2015
Everyone, enjoy! :)

Note that we do have (and use) thread annotations in webrtc/base/thread_annotations.h to compile-time enforce that locks are taken. These seem to be enough for us in most cases. I think we can close this bug unless you disagree.

In the example fischman@ pointed to HandleOverrun can be annotated as requiring a lock to be taken which would've solved this in compile-time (better than runtime asserts).
Project Member Comment 4 by, Nov 8 2016
Labels: Pri-3
Project Member Comment 5 by, Dec 14 2016
Owner: ----
Project Member Comment 6 by, Sep 19
Status: Archived
Sign in to add a comment