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

Commit dabf892a authored by jiabin's avatar jiabin
Browse files

Remove unnecessary FIXME in close. 

Close is already synchronous after adding command queue. Currently, the
close function will only called when opening error or from command
queue.

Bug: 224853525
Test: make
Change-Id: If9a2ed0af7d926046ad3f764095b37aba47ef6a6
parent 14d0ae53

services/oboeservice/AAudioServiceEndpointMMAP.cpp

+0 −3
Original line number Diff line number Diff line
@@ -228,9 +228,6 @@ void AAudioServiceEndpointMMAP::close() { 
    if (mMmapStream != nullptr) {

        // Needs to be explicitly cleared or CTS will fail but it is not clear why.

        mMmapStream.clear();

        // Apparently the above close is asynchronous. An attempt to open a new device

        // right after a close can fail. Also some callbacks may still be in flight!

        // FIXME Make closing synchronous.

        AudioClock::sleepForNanos(100 * AAUDIO_NANOS_PER_MILLISECOND);

    }

}