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

Unified Diff: Source/platform/audio/android/FFTFrameOpenMAXDLAndroid.cpp

Issue 536843002: Change members's order and combine duplicated method for FFTFrame. (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master
Patch Set: Separated using VectorMatch::vsmul from FFTFrameMac.cpp Created 6 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « Source/platform/audio/FFTFrameStub.cpp ('k') | Source/platform/audio/ffmpeg/FFTFrameFFMPEG.cpp » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: Source/platform/audio/android/FFTFrameOpenMAXDLAndroid.cpp
diff --git a/Source/platform/audio/android/FFTFrameOpenMAXDLAndroid.cpp b/Source/platform/audio/android/FFTFrameOpenMAXDLAndroid.cpp
index d4b01fc738732c6d23cd4ca5de7497e4f0629c1a..b4abb2ae399b372604cb673ff6779b847ceebe95 100644
--- a/Source/platform/audio/android/FFTFrameOpenMAXDLAndroid.cpp
+++ b/Source/platform/audio/android/FFTFrameOpenMAXDLAndroid.cpp
@@ -45,11 +45,11 @@ const unsigned kMaxFFTPow2Size = 15;
FFTFrame::FFTFrame(unsigned fftSize)
: m_FFTSize(fftSize)
, m_log2FFTSize(static_cast<unsigned>(log2(fftSize)))
+ , m_realData(fftSize / 2)
+ , m_imagData(fftSize / 2)
, m_forwardContext(0)
, m_inverseContext(0)
, m_complexData(fftSize)
- , m_realData(fftSize / 2)
- , m_imagData(fftSize / 2)
{
// We only allow power of two.
ASSERT(1UL << m_log2FFTSize == m_FFTSize);
@@ -71,11 +71,11 @@ FFTFrame::FFTFrame()
FFTFrame::FFTFrame(const FFTFrame& frame)
: m_FFTSize(frame.m_FFTSize)
, m_log2FFTSize(frame.m_log2FFTSize)
+ , m_realData(frame.m_FFTSize / 2)
+ , m_imagData(frame.m_FFTSize / 2)
, m_forwardContext(0)
, m_inverseContext(0)
, m_complexData(frame.m_FFTSize)
- , m_realData(frame.m_FFTSize / 2)
- , m_imagData(frame.m_FFTSize / 2)
{
m_forwardContext = contextForSize(m_log2FFTSize);
m_inverseContext = contextForSize(m_log2FFTSize);
@@ -154,16 +154,6 @@ void FFTFrame::doInverseFFT(float* data)
}
}
-float* FFTFrame::realData() const
-{
- return const_cast<float*>(m_realData.data());
-}
-
-float* FFTFrame::imagData() const
-{
- return const_cast<float*>(m_imagData.data());
-}
-
OMXFFTSpec_R_F32* FFTFrame::contextForSize(unsigned log2FFTSize)
{
ASSERT(log2FFTSize);
« no previous file with comments | « Source/platform/audio/FFTFrameStub.cpp ('k') | Source/platform/audio/ffmpeg/FFTFrameFFMPEG.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698