commit d510fddbaf582b82fd6ed5add5081234ce7807e7
author Harish Mahendrakar <harish.mahendrakar@ittiam.com> Tue Mar 16 13:26:31 2021 -0700
committer Harish Mahendrakar <harish.mahendrakar@ittiam.com> Wed Mar 17 23:10:18 2021 +0000
tree bd165d57fc7dd1ff7cd2cffc0fbe07e8be2c2620
parent 0a2021b06fa5409d4c5b6a21a81d4f0dc6c37d9c

libaudiopreprocessing: Cleanup PreProcessing.cpp

- Removed unused elements from preproc_session_s
- Fixed initialization of frameCount in PreProcessingTest.cpp
- Print number of frames processed in PreProcessingTest.cpp

Bug: 182583445
Test: atest EffectPreprocessingTest
Test: preprocessing/tests/build_and_run_all_unit_tests.sh

Change-Id: I4a8fb5796451eb2b3a37d6e4bd48452bccac9318

media/libeffects/preprocessing/tests/PreProcessingTest.cpp

diff --git a/media/libeffects/preprocessing/tests/PreProcessingTest.cpp b/media/libeffects/preprocessing/tests/PreProcessingTest.cpp
index e0025fe..3bd93f8 100644
--- a/media/libeffects/preprocessing/tests/PreProcessingTest.cpp
+++ b/media/libeffects/preprocessing/tests/PreProcessingTest.cpp
@@ -451,8 +451,8 @@
         }
         audio_buffer_t inputBuffer, outputBuffer;
         audio_buffer_t farInBuffer{};
-        inputBuffer.frameCount = samplesRead;
-        outputBuffer.frameCount = samplesRead;
+        inputBuffer.frameCount = frameLength;
+        outputBuffer.frameCount = frameLength;
         inputBuffer.s16 = in.data();
         outputBuffer.s16 = out.data();
 
@@ -472,7 +472,7 @@
                 }
             }
 
-            farInBuffer.frameCount = samplesRead;
+            farInBuffer.frameCount = frameLength;
             farInBuffer.s16 = farIn.data();
         }
 
@@ -519,6 +519,7 @@
         }
         frameCounter += frameLength;
     }
+    printf("frameCounter: [%d]\n", frameCounter);
     // Release all the effect handles created
     for (int i = 0; i < PREPROC_NUM_EFFECTS; i++) {
         if (int status = AUDIO_EFFECT_LIBRARY_INFO_SYM.release_effect(effectHandle[i]);