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

Unified Diff: third_party/WebKit/Source/platform/audio/Biquad.cpp

Issue 2384073002: reflow comments in platform/audio (Closed)
Patch Set: comments (heh!) Created 4 years, 2 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 side-by-side diff with in-line comments
Download patch
Index: third_party/WebKit/Source/platform/audio/Biquad.cpp
diff --git a/third_party/WebKit/Source/platform/audio/Biquad.cpp b/third_party/WebKit/Source/platform/audio/Biquad.cpp
index 7297db2f1aff09f9eb06bea4cd7a763bcacff148..9d4c636c4aa958075c041bc2584b4f28084847ff 100644
--- a/third_party/WebKit/Source/platform/audio/Biquad.cpp
+++ b/third_party/WebKit/Source/platform/audio/Biquad.cpp
@@ -52,8 +52,8 @@ Biquad::Biquad() : m_hasSampleAccurateValues(false) {
m_outputBuffer.allocate(kBufferSize + 2);
#endif
- // Allocate enough space for the a-rate filter coefficients to handle a rendering quantum of 128
- // frames.
+ // Allocate enough space for the a-rate filter coefficients to handle a
+ // rendering quantum of 128 frames.
m_b0.allocate(AudioUtilities::kRenderQuantumFrames);
m_b1.allocate(AudioUtilities::kRenderQuantumFrames);
m_b2.allocate(AudioUtilities::kRenderQuantumFrames);
@@ -109,14 +109,14 @@ void Biquad::process(const float* sourceP,
m_y1 = DenormalDisabler::flushDenormalFloatToZero(y1);
m_y2 = DenormalDisabler::flushDenormalFloatToZero(y2);
- // There is an assumption here that once we have sample accurate values we can never go back
- // to not having sample accurate values. This is currently true in the way
- // AudioParamTimline is implemented: once an event is inserted, sample accurate processing
- // is always enabled.
+ // There is an assumption here that once we have sample accurate values we
+ // can never go back to not having sample accurate values. This is
+ // currently true in the way AudioParamTimline is implemented: once an
+ // event is inserted, sample accurate processing is always enabled.
//
- // If so, then we never have to update the state variables for the MACOSX path. The
- // structure of the state variable in these cases aren't well documented so it's not clear
- // how to update them anyway.
+ // If so, then we never have to update the state variables for the MACOSX
+ // path. The structure of the state variable in these cases aren't well
+ // documented so it's not clear how to update them anyway.
} else {
#if OS(MACOSX)
double* inputP = m_inputBuffer.data();
@@ -231,9 +231,10 @@ void Biquad::processSliceFast(double* sourceP,
// Use double-precision for filter stability
vDSP_deq22D(sourceP, 1, coefficientsP, destP, 1, framesToProcess);
- // Save history. Note that sourceP and destP reference m_inputBuffer and m_outputBuffer respectively.
- // These buffers are allocated (in the constructor) with space for two extra samples so it's OK to access
- // array values two beyond framesToProcess.
+ // Save history. Note that sourceP and destP reference m_inputBuffer and
+ // m_outputBuffer respectively. These buffers are allocated (in the
+ // constructor) with space for two extra samples so it's OK to access array
+ // values two beyond framesToProcess.
sourceP[0] = sourceP[framesToProcess - 2 + 2];
sourceP[1] = sourceP[framesToProcess - 1 + 2];
destP[0] = destP[framesToProcess - 2 + 2];
« no previous file with comments | « third_party/WebKit/Source/platform/audio/Biquad.h ('k') | third_party/WebKit/Source/platform/audio/DenormalDisabler.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698