Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 2bec8325 authored by Andy Hung's avatar Andy Hung Committed by Jessica Wagantall
Browse files

DO NOT MERGE - AudioFlinger: Clear record buffers when starting RecordThread 

Ticket: CYNGNOS-1299
Bug: 24211743
Bug: 24267152
Change-Id: I58c55e56b85067b71e4e300f947b4dfc159637ba
(cherry picked from commit 8c987fa7)
parent bb253fcc

services/audioflinger/FastCapture.cpp

+1 −0
Original line number Diff line number Diff line
@@ -134,6 +134,7 @@ void FastCapture::onStateChange() 
            unsigned channelCount = Format_channelCount(format);

            // FIXME frameSize

            readBuffer = new short[frameCount * channelCount];

            memset(readBuffer, 0, frameCount * channelCount * sizeof(readBuffer[0]));

            periodNs = (frameCount * 1000000000LL) / sampleRate;    // 1.00

            underrunNs = (frameCount * 1750000000LL) / sampleRate;  // 1.75

            overrunNs = (frameCount * 500000000LL) / sampleRate;    // 0.50

services/audioflinger/Threads.cpp

+3 −1
Original line number Diff line number Diff line
@@ -6682,7 +6682,9 @@ void AudioFlinger::RecordThread::readInputParameters_l() 
    // The current value is higher than necessary.  However it should not add to latency.



    // Over-allocate beyond mRsmpInFramesP2 to permit a HAL read past end of buffer

    mRsmpInBuffer = new int16_t[(mRsmpInFramesP2 + mFrameCount - 1) * mChannelCount];

    size_t bufferSizeInShorts = (mRsmpInFramesP2 + mFrameCount - 1) * mChannelCount;

    mRsmpInBuffer = new int16_t[bufferSizeInShorts];

    memset(mRsmpInBuffer, 0, bufferSizeInShorts * sizeof(mRsmpInBuffer[0]));



    // AudioRecord mSampleRate and mChannelCount are constant due to AudioRecord API constraints.

    // But if thread's mSampleRate or mChannelCount changes, how will that affect active tracks?