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

Side by Side Diff: webrtc/modules/audio_processing/audio_processing_impl.cc

Issue 2288153002: Fix Chromium clang plugin warnings (Closed)
Patch Set: Created 4 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
(...skipping 260 matching lines...) Expand 10 before | Expand all | Expand 10 after
271 int AudioProcessingImpl::MaybeInitializeRender( 271 int AudioProcessingImpl::MaybeInitializeRender(
272 const ProcessingConfig& processing_config) { 272 const ProcessingConfig& processing_config) {
273 return MaybeInitialize(processing_config); 273 return MaybeInitialize(processing_config);
274 } 274 }
275 275
276 int AudioProcessingImpl::MaybeInitializeCapture( 276 int AudioProcessingImpl::MaybeInitializeCapture(
277 const ProcessingConfig& processing_config) { 277 const ProcessingConfig& processing_config) {
278 return MaybeInitialize(processing_config); 278 return MaybeInitialize(processing_config);
279 } 279 }
280 280
281 #ifdef WEBRTC_AUDIOPROC_DEBUG_DUMP
282
283 AudioProcessingImpl::ApmDebugDumpThreadState::ApmDebugDumpThreadState()
284 : event_msg(new audioproc::Event()) {}
285
286 AudioProcessingImpl::ApmDebugDumpThreadState::~ApmDebugDumpThreadState() {}
287
288 AudioProcessingImpl::ApmDebugDumpState::ApmDebugDumpState()
289 : debug_file(FileWrapper::Create()) {}
290
291 AudioProcessingImpl::ApmDebugDumpState::~ApmDebugDumpState() {}
292
293 #endif // WEBRTC_AUDIOPROC_DEBUG_DUMP
294
281 // Calls InitializeLocked() if any of the audio parameters have changed from 295 // Calls InitializeLocked() if any of the audio parameters have changed from
282 // their current values (needs to be called while holding the crit_render_lock). 296 // their current values (needs to be called while holding the crit_render_lock).
283 int AudioProcessingImpl::MaybeInitialize( 297 int AudioProcessingImpl::MaybeInitialize(
284 const ProcessingConfig& processing_config) { 298 const ProcessingConfig& processing_config) {
285 // Called from both threads. Thread check is therefore not possible. 299 // Called from both threads. Thread check is therefore not possible.
286 if (processing_config == formats_.api_format) { 300 if (processing_config == formats_.api_format) {
287 return kNoError; 301 return kNoError;
288 } 302 }
289 303
290 rtc::CritScope cs_capture(&crit_capture_); 304 rtc::CritScope cs_capture(&crit_capture_);
(...skipping 1226 matching lines...) Expand 10 before | Expand all | Expand 10 after
1517 debug_dump_.capture.event_msg->set_type(audioproc::Event::CONFIG); 1531 debug_dump_.capture.event_msg->set_type(audioproc::Event::CONFIG);
1518 debug_dump_.capture.event_msg->mutable_config()->CopyFrom(config); 1532 debug_dump_.capture.event_msg->mutable_config()->CopyFrom(config);
1519 1533
1520 RETURN_ON_ERR(WriteMessageToDebugFile(debug_dump_.debug_file.get(), 1534 RETURN_ON_ERR(WriteMessageToDebugFile(debug_dump_.debug_file.get(),
1521 &debug_dump_.num_bytes_left_for_log_, 1535 &debug_dump_.num_bytes_left_for_log_,
1522 &crit_debug_, &debug_dump_.capture)); 1536 &crit_debug_, &debug_dump_.capture));
1523 return kNoError; 1537 return kNoError;
1524 } 1538 }
1525 #endif // WEBRTC_AUDIOPROC_DEBUG_DUMP 1539 #endif // WEBRTC_AUDIOPROC_DEBUG_DUMP
1526 1540
1541 AudioProcessingImpl::ApmCaptureState::ApmCaptureState(
1542 bool transient_suppressor_enabled,
1543 const std::vector<Point>& array_geometry,
1544 SphericalPointf target_direction)
1545 : aec_system_delay_jumps(-1),
1546 delay_offset_ms(0),
1547 was_stream_delay_set(false),
1548 last_stream_delay_ms(0),
1549 last_aec_system_delay_ms(0),
1550 stream_delay_jumps(-1),
1551 output_will_be_muted(false),
1552 key_pressed(false),
1553 transient_suppressor_enabled(transient_suppressor_enabled),
1554 array_geometry(array_geometry),
1555 target_direction(target_direction),
1556 fwd_proc_format(kSampleRate16kHz),
1557 split_rate(kSampleRate16kHz) {}
1558
1559 AudioProcessingImpl::ApmCaptureState::~ApmCaptureState() = default;
1560
1561 AudioProcessingImpl::ApmRenderState::ApmRenderState() = default;
1562
1563 AudioProcessingImpl::ApmRenderState::~ApmRenderState() = default;
1564
1527 } // namespace webrtc 1565 } // namespace webrtc
OLDNEW
« no previous file with comments | « webrtc/modules/audio_processing/audio_processing_impl.h ('k') | webrtc/modules/audio_processing/echo_cancellation_impl.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698