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

Unified Diff: third_party/WebKit/Source/platform/audio/ReverbConvolver.h

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/ReverbConvolver.h
diff --git a/third_party/WebKit/Source/platform/audio/ReverbConvolver.h b/third_party/WebKit/Source/platform/audio/ReverbConvolver.h
index e4fc69a6232c6b24d0381ee6b2a04a53cfa0c1c5..2e37dc29630dea1f87988f74d17823a40a8d1f9f 100644
--- a/third_party/WebKit/Source/platform/audio/ReverbConvolver.h
+++ b/third_party/WebKit/Source/platform/audio/ReverbConvolver.h
@@ -49,10 +49,11 @@ class PLATFORM_EXPORT ReverbConvolver {
WTF_MAKE_NONCOPYABLE(ReverbConvolver);
public:
- // maxFFTSize can be adjusted (from say 2048 to 32768) depending on how much precision is necessary.
- // For certain tweaky de-convolving applications the phase errors add up quickly and lead to non-sensical results with
- // larger FFT sizes and single-precision floats. In these cases 2048 is a good size.
- // If not doing multi-threaded convolution, then should not go > 8192.
+ // maxFFTSize can be adjusted (from say 2048 to 32768) depending on how much
+ // precision is necessary. For certain tweaky de-convolving applications the
+ // phase errors add up quickly and lead to non-sensical results with larger
+ // FFT sizes and single-precision floats. In these cases 2048 is a good
+ // size. If not doing multi-threaded convolution, then should not go > 8192.
ReverbConvolver(AudioChannel* impulseResponse,
size_t renderSliceSize,
size_t maxFFTSize,
@@ -78,14 +79,17 @@ class PLATFORM_EXPORT ReverbConvolver {
ReverbAccumulationBuffer m_accumulationBuffer;
- // One or more background threads read from this input buffer which is fed from the realtime thread.
+ // One or more background threads read from this input buffer which is fed
+ // from the realtime thread.
ReverbInputBuffer m_inputBuffer;
- // First stage will be of size m_minFFTSize. Each next stage will be twice as big until we hit m_maxFFTSize.
+ // First stage will be of size m_minFFTSize. Each next stage will be twice as
+ // big until we hit m_maxFFTSize.
size_t m_minFFTSize;
size_t m_maxFFTSize;
- // But don't exceed this size in the real-time thread (if we're doing background processing).
+ // But don't exceed this size in the real-time thread (if we're doing
+ // background processing).
size_t m_maxRealtimeFFTSize;
// Background thread and synchronization

Powered by Google App Engine
This is Rietveld 408576698