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

Side by Side Diff: webrtc/modules/audio_device/audio_device_impl.cc

Issue 1703833002: Remove ignored return code from modules. (Closed) Base URL: https://chromium.googlesource.com/external/webrtc.git@master
Patch Set: rebase Created 4 years, 10 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
(...skipping 409 matching lines...) Expand 10 before | Expand all | Expand 10 after
420 return deltaProcess; 420 return deltaProcess;
421 } 421 }
422 422
423 // ---------------------------------------------------------------------------- 423 // ----------------------------------------------------------------------------
424 // Module::Process 424 // Module::Process
425 // 425 //
426 // Check for posted error and warning reports. Generate callbacks if 426 // Check for posted error and warning reports. Generate callbacks if
427 // new reports exists. 427 // new reports exists.
428 // ---------------------------------------------------------------------------- 428 // ----------------------------------------------------------------------------
429 429
430 int32_t AudioDeviceModuleImpl::Process() 430 void AudioDeviceModuleImpl::Process()
431 { 431 {
432 432
433 _lastProcessTime = TickTime::MillisecondTimestamp(); 433 _lastProcessTime = TickTime::MillisecondTimestamp();
434 434
435 // kPlayoutWarning 435 // kPlayoutWarning
436 if (_ptrAudioDevice->PlayoutWarning()) 436 if (_ptrAudioDevice->PlayoutWarning())
437 { 437 {
438 CriticalSectionScoped lock(&_critSectEventCb); 438 CriticalSectionScoped lock(&_critSectEventCb);
439 if (_ptrCbAudioDeviceObserver) 439 if (_ptrCbAudioDeviceObserver)
440 { 440 {
(...skipping 31 matching lines...) Expand 10 before | Expand all | Expand 10 after
472 if (_ptrAudioDevice->RecordingError()) 472 if (_ptrAudioDevice->RecordingError())
473 { 473 {
474 CriticalSectionScoped lock(&_critSectEventCb); 474 CriticalSectionScoped lock(&_critSectEventCb);
475 if (_ptrCbAudioDeviceObserver) 475 if (_ptrCbAudioDeviceObserver)
476 { 476 {
477 WEBRTC_TRACE(kTraceError, kTraceAudioDevice, _id, "=> OnErrorIsRepor ted(kRecordingError)"); 477 WEBRTC_TRACE(kTraceError, kTraceAudioDevice, _id, "=> OnErrorIsRepor ted(kRecordingError)");
478 _ptrCbAudioDeviceObserver->OnErrorIsReported(AudioDeviceObserver::kR ecordingError); 478 _ptrCbAudioDeviceObserver->OnErrorIsReported(AudioDeviceObserver::kR ecordingError);
479 } 479 }
480 _ptrAudioDevice->ClearRecordingError(); 480 _ptrAudioDevice->ClearRecordingError();
481 } 481 }
482
483 return 0;
484 } 482 }
485 483
486 // ============================================================================ 484 // ============================================================================
487 // Public API 485 // Public API
488 // ============================================================================ 486 // ============================================================================
489 487
490 // ---------------------------------------------------------------------------- 488 // ----------------------------------------------------------------------------
491 // ActiveAudioLayer 489 // ActiveAudioLayer
492 // ---------------------------------------------------------------------------- 490 // ----------------------------------------------------------------------------
493 491
(...skipping 1444 matching lines...) Expand 10 before | Expand all | Expand 10 after
1938 // ---------------------------------------------------------------------------- 1936 // ----------------------------------------------------------------------------
1939 // PlatformAudioLayer 1937 // PlatformAudioLayer
1940 // ---------------------------------------------------------------------------- 1938 // ----------------------------------------------------------------------------
1941 1939
1942 AudioDeviceModule::AudioLayer AudioDeviceModuleImpl::PlatformAudioLayer() const 1940 AudioDeviceModule::AudioLayer AudioDeviceModuleImpl::PlatformAudioLayer() const
1943 { 1941 {
1944 return _platformAudioLayer; 1942 return _platformAudioLayer;
1945 } 1943 }
1946 1944
1947 } // namespace webrtc 1945 } // namespace webrtc
OLDNEW
« no previous file with comments | « webrtc/modules/audio_device/audio_device_impl.h ('k') | webrtc/modules/audio_device/include/fake_audio_device.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698