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

Commit bf8edbe0 authored by Treehugger Robot's avatar Treehugger Robot Committed by Automerger Merge Worker
Browse files

Merge "BufferQueueProducer: Log verbose log to warning instead of error." am: e9b2010f 

Original change: https://android-review.googlesource.com/c/platform/frameworks/native/+/1822354

Change-Id: If3a55b34bea11174edf9127c8d2b6ca6774f6c82
parents bd0fef63 e9b2010f

libs/gui/BufferQueueProducer.cpp

+4 −1
Original line number Diff line number Diff line
@@ -645,7 +645,10 @@ status_t BufferQueueProducer::detachBuffer(int slot) { 
                    slot, BufferQueueDefs::NUM_BUFFER_SLOTS);

            return BAD_VALUE;

        } else if (!mSlots[slot].mBufferState.isDequeued()) {

            BQ_LOGE("detachBuffer: slot %d is not owned by the producer "

            // TODO(http://b/140581935): This message is BQ_LOGW because it

            // often logs when no actionable errors are present. Return to

            // using BQ_LOGE after ensuring this only logs during errors.

            BQ_LOGW("detachBuffer: slot %d is not owned by the producer "

                    "(state = %s)", slot, mSlots[slot].mBufferState.string());

            return BAD_VALUE;

        } else if (!mSlots[slot].mRequestBufferCalled) {