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

Side by Side Diff: webrtc/common_audio/sparse_fir_filter.cc

Issue 2288133002: 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
« no previous file with comments | « webrtc/common_audio/sparse_fir_filter.h ('k') | webrtc/modules/audio_processing/BUILD.gn » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
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
11 #include "webrtc/common_audio/sparse_fir_filter.h" 11 #include "webrtc/common_audio/sparse_fir_filter.h"
12 12
13 #include "webrtc/base/checks.h" 13 #include "webrtc/base/checks.h"
14 14
15 namespace webrtc { 15 namespace webrtc {
16 16
17 SparseFIRFilter::SparseFIRFilter(const float* nonzero_coeffs, 17 SparseFIRFilter::SparseFIRFilter(const float* nonzero_coeffs,
18 size_t num_nonzero_coeffs, 18 size_t num_nonzero_coeffs,
19 size_t sparsity, 19 size_t sparsity,
20 size_t offset) 20 size_t offset)
21 : sparsity_(sparsity), 21 : sparsity_(sparsity),
22 offset_(offset), 22 offset_(offset),
23 nonzero_coeffs_(nonzero_coeffs, nonzero_coeffs + num_nonzero_coeffs), 23 nonzero_coeffs_(nonzero_coeffs, nonzero_coeffs + num_nonzero_coeffs),
24 state_(sparsity_ * (num_nonzero_coeffs - 1) + offset_, 0.f) { 24 state_(sparsity_ * (num_nonzero_coeffs - 1) + offset_, 0.f) {
25 RTC_CHECK_GE(num_nonzero_coeffs, 1u); 25 RTC_CHECK_GE(num_nonzero_coeffs, 1u);
26 RTC_CHECK_GE(sparsity, 1u); 26 RTC_CHECK_GE(sparsity, 1u);
27 } 27 }
28 28
29 SparseFIRFilter::~SparseFIRFilter() = default;
30
29 void SparseFIRFilter::Filter(const float* in, size_t length, float* out) { 31 void SparseFIRFilter::Filter(const float* in, size_t length, float* out) {
30 // Convolves the input signal |in| with the filter kernel |nonzero_coeffs_| 32 // Convolves the input signal |in| with the filter kernel |nonzero_coeffs_|
31 // taking into account the previous state. 33 // taking into account the previous state.
32 for (size_t i = 0; i < length; ++i) { 34 for (size_t i = 0; i < length; ++i) {
33 out[i] = 0.f; 35 out[i] = 0.f;
34 size_t j; 36 size_t j;
35 for (j = 0; i >= j * sparsity_ + offset_ && 37 for (j = 0; i >= j * sparsity_ + offset_ &&
36 j < nonzero_coeffs_.size(); ++j) { 38 j < nonzero_coeffs_.size(); ++j) {
37 out[i] += in[i - j * sparsity_ - offset_] * nonzero_coeffs_[j]; 39 out[i] += in[i - j * sparsity_ - offset_] * nonzero_coeffs_[j];
38 } 40 }
(...skipping 12 matching lines...) Expand all
51 } else { 53 } else {
52 std::memmove(&state_[0], 54 std::memmove(&state_[0],
53 &state_[length], 55 &state_[length],
54 (state_.size() - length) * sizeof(state_[0])); 56 (state_.size() - length) * sizeof(state_[0]));
55 std::memcpy(&state_[state_.size() - length], in, length * sizeof(*in)); 57 std::memcpy(&state_[state_.size() - length], in, length * sizeof(*in));
56 } 58 }
57 } 59 }
58 } 60 }
59 61
60 } // namespace webrtc 62 } // namespace webrtc
OLDNEW
« no previous file with comments | « webrtc/common_audio/sparse_fir_filter.h ('k') | webrtc/modules/audio_processing/BUILD.gn » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698