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

Side by Side Diff: webrtc/modules/audio_device/android/opensles_player.cc

Issue 2535593002: RTC_[D]CHECK_op: Remove "u" suffix on integer constants (Closed)
Patch Set: Created 4 years 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) 2015 The WebRTC project authors. All Rights Reserved. 2 * Copyright (c) 2015 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
(...skipping 127 matching lines...) Expand 10 before | Expand all | Expand 10 after
138 return 0; 138 return 0;
139 } 139 }
140 // Stop playing by setting the play state to SL_PLAYSTATE_STOPPED. 140 // Stop playing by setting the play state to SL_PLAYSTATE_STOPPED.
141 RETURN_ON_ERROR((*player_)->SetPlayState(player_, SL_PLAYSTATE_STOPPED), -1); 141 RETURN_ON_ERROR((*player_)->SetPlayState(player_, SL_PLAYSTATE_STOPPED), -1);
142 // Clear the buffer queue to flush out any remaining data. 142 // Clear the buffer queue to flush out any remaining data.
143 RETURN_ON_ERROR((*simple_buffer_queue_)->Clear(simple_buffer_queue_), -1); 143 RETURN_ON_ERROR((*simple_buffer_queue_)->Clear(simple_buffer_queue_), -1);
144 #if RTC_DCHECK_IS_ON 144 #if RTC_DCHECK_IS_ON
145 // Verify that the buffer queue is in fact cleared as it should. 145 // Verify that the buffer queue is in fact cleared as it should.
146 SLAndroidSimpleBufferQueueState buffer_queue_state; 146 SLAndroidSimpleBufferQueueState buffer_queue_state;
147 (*simple_buffer_queue_)->GetState(simple_buffer_queue_, &buffer_queue_state); 147 (*simple_buffer_queue_)->GetState(simple_buffer_queue_, &buffer_queue_state);
148 RTC_DCHECK_EQ(0u, buffer_queue_state.count); 148 RTC_DCHECK_EQ(0, buffer_queue_state.count);
149 RTC_DCHECK_EQ(0u, buffer_queue_state.index); 149 RTC_DCHECK_EQ(0, buffer_queue_state.index);
150 #endif 150 #endif
151 // The number of lower latency audio players is limited, hence we create the 151 // The number of lower latency audio players is limited, hence we create the
152 // audio player in Start() and destroy it in Stop(). 152 // audio player in Start() and destroy it in Stop().
153 DestroyAudioPlayer(); 153 DestroyAudioPlayer();
154 thread_checker_opensles_.DetachFromThread(); 154 thread_checker_opensles_.DetachFromThread();
155 initialized_ = false; 155 initialized_ = false;
156 playing_ = false; 156 playing_ = false;
157 return 0; 157 return 0;
158 } 158 }
159 159
(...skipping 257 matching lines...) Expand 10 before | Expand all | Expand 10 after
417 RTC_DCHECK(player_); 417 RTC_DCHECK(player_);
418 SLuint32 state; 418 SLuint32 state;
419 SLresult err = (*player_)->GetPlayState(player_, &state); 419 SLresult err = (*player_)->GetPlayState(player_, &state);
420 if (SL_RESULT_SUCCESS != err) { 420 if (SL_RESULT_SUCCESS != err) {
421 ALOGE("GetPlayState failed: %d", err); 421 ALOGE("GetPlayState failed: %d", err);
422 } 422 }
423 return state; 423 return state;
424 } 424 }
425 425
426 } // namespace webrtc 426 } // namespace webrtc
OLDNEW
« no previous file with comments | « webrtc/modules/audio_coding/neteq/tools/neteq_replacement_input.cc ('k') | webrtc/modules/audio_device/audio_device_buffer.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698