Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(201)

Side by Side Diff: webrtc/system_wrappers/source/critical_section_posix.cc

Issue 1335923002: Add RTC_ prefix to (D)CHECKs and related macros. (Closed) Base URL: https://chromium.googlesource.com/external/webrtc.git@master
Patch Set: Rebase. Created 5 years, 3 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
1 /* 1 /*
2 * Copyright (c) 2012 The WebRTC project authors. All Rights Reserved. 2 * Copyright (c) 2012 The WebRTC project authors. All Rights Reserved.
3 * 3 *
4 * Use of this source code is governed by a BSD-style license 4 * Use of this source code is governed by a BSD-style license
5 * that can be found in the LICENSE file in the root of the source 5 * that can be found in the LICENSE file in the root of the source
6 * tree. An additional intellectual property rights grant can be found 6 * tree. An additional intellectual property rights grant can be found
7 * in the file PATENTS. All contributing project authors may 7 * in the file PATENTS. All contributing project authors may
8 * be found in the AUTHORS file in the root of the source tree. 8 * be found in the AUTHORS file in the root of the source tree.
9 */ 9 */
10 10
11 // General note: return values for the various pthread synchronization APIs 11 // General note: return values for the various pthread synchronization APIs
12 // are explicitly ignored here. In Chromium, the same thing is done for release. 12 // are explicitly ignored here. In Chromium, the same thing is done for release.
13 // However, in debugging, failure in these APIs are logged. There is currently 13 // However, in debugging, failure in these APIs are logged.
14 // no equivalent to DCHECK_EQ in WebRTC code so this is the best we can do here.
15 // TODO(henrike): add logging when pthread synchronization APIs are failing. 14 // TODO(henrike): add logging when pthread synchronization APIs are failing.
16 15
17 #include "webrtc/system_wrappers/source/critical_section_posix.h" 16 #include "webrtc/system_wrappers/source/critical_section_posix.h"
18 17
19 namespace webrtc { 18 namespace webrtc {
20 19
21 CriticalSectionPosix::CriticalSectionPosix() { 20 CriticalSectionPosix::CriticalSectionPosix() {
22 pthread_mutexattr_t attr; 21 pthread_mutexattr_t attr;
23 (void) pthread_mutexattr_init(&attr); 22 (void) pthread_mutexattr_init(&attr);
24 (void) pthread_mutexattr_settype(&attr, PTHREAD_MUTEX_RECURSIVE); 23 (void) pthread_mutexattr_settype(&attr, PTHREAD_MUTEX_RECURSIVE);
25 (void) pthread_mutex_init(&mutex_, &attr); 24 (void) pthread_mutex_init(&mutex_, &attr);
26 } 25 }
27 26
28 CriticalSectionPosix::~CriticalSectionPosix() { 27 CriticalSectionPosix::~CriticalSectionPosix() {
29 (void) pthread_mutex_destroy(&mutex_); 28 (void) pthread_mutex_destroy(&mutex_);
30 } 29 }
31 30
32 void 31 void
33 CriticalSectionPosix::Enter() { 32 CriticalSectionPosix::Enter() {
34 (void) pthread_mutex_lock(&mutex_); 33 (void) pthread_mutex_lock(&mutex_);
35 } 34 }
36 35
37 void 36 void
38 CriticalSectionPosix::Leave() { 37 CriticalSectionPosix::Leave() {
39 (void) pthread_mutex_unlock(&mutex_); 38 (void) pthread_mutex_unlock(&mutex_);
40 } 39 }
41 40
42 } // namespace webrtc 41 } // namespace webrtc
OLDNEW
« no previous file with comments | « webrtc/system_wrappers/interface/scoped_vector.h ('k') | webrtc/system_wrappers/source/event_timer_posix.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698