Change DCHECK( == 0) to DCHECK_EQ in the posix lock implementation.
It is helpful to see the actual value during a check failure. Review URL: http://codereview.chromium.org/42225 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@11741 0039d316-1c4b-4281-b951-d872f2087c98
Esse commit está contido em:
@@ -13,13 +13,13 @@ LockImpl::LockImpl() {
|
||||
// In debug, setup attributes for lock error checking.
|
||||
pthread_mutexattr_t mta;
|
||||
int rv = pthread_mutexattr_init(&mta);
|
||||
DCHECK(rv == 0);
|
||||
DCHECK_EQ(rv, 0);
|
||||
rv = pthread_mutexattr_settype(&mta, PTHREAD_MUTEX_ERRORCHECK);
|
||||
DCHECK(rv == 0);
|
||||
DCHECK_EQ(rv, 0);
|
||||
rv = pthread_mutex_init(&os_lock_, &mta);
|
||||
DCHECK(rv == 0);
|
||||
DCHECK_EQ(rv, 0);
|
||||
rv = pthread_mutexattr_destroy(&mta);
|
||||
DCHECK(rv == 0);
|
||||
DCHECK_EQ(rv, 0);
|
||||
#else
|
||||
// In release, go with the default lock attributes.
|
||||
pthread_mutex_init(&os_lock_, NULL);
|
||||
@@ -28,7 +28,7 @@ LockImpl::LockImpl() {
|
||||
|
||||
LockImpl::~LockImpl() {
|
||||
int rv = pthread_mutex_destroy(&os_lock_);
|
||||
DCHECK(rv == 0);
|
||||
DCHECK_EQ(rv, 0);
|
||||
}
|
||||
|
||||
bool LockImpl::Try() {
|
||||
@@ -39,10 +39,10 @@ bool LockImpl::Try() {
|
||||
|
||||
void LockImpl::Lock() {
|
||||
int rv = pthread_mutex_lock(&os_lock_);
|
||||
DCHECK(rv == 0);
|
||||
DCHECK_EQ(rv, 0);
|
||||
}
|
||||
|
||||
void LockImpl::Unlock() {
|
||||
int rv = pthread_mutex_unlock(&os_lock_);
|
||||
DCHECK(rv == 0);
|
||||
DCHECK_EQ(rv, 0);
|
||||
}
|
||||
|
||||
Referência em uma Nova Issue
Bloquear um usuário