summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
-rw-r--r--libs/gui/ConsumerBase.cpp34
1 files changed, 10 insertions, 24 deletions
diff --git a/libs/gui/ConsumerBase.cpp b/libs/gui/ConsumerBase.cpp
index cfc0293..624d7e0 100644
--- a/libs/gui/ConsumerBase.cpp
+++ b/libs/gui/ConsumerBase.cpp
@@ -109,35 +109,21 @@ void ConsumerBase::onFrameAvailable() {
}
void ConsumerBase::onBuffersReleased() {
- sp<GraphicBuffer> bufRefs[BufferQueue::NUM_BUFFER_SLOTS];
-
- { // Scope for the lock
- Mutex::Autolock lock(mMutex);
-
- CB_LOGV("onBuffersReleased");
-
- if (mAbandoned) {
- // Nothing to do if we're already abandoned.
- return;
- }
+ Mutex::Autolock lock(mMutex);
- uint32_t mask = 0;
- mBufferQueue->getReleasedBuffers(&mask);
- for (int i = 0; i < BufferQueue::NUM_BUFFER_SLOTS; i++) {
- if (mask & (1 << i)) {
- // Grab a local reference to the buffers so that they don't
- // get freed while the lock is held.
- bufRefs[i] = mSlots[i].mGraphicBuffer;
+ CB_LOGV("onBuffersReleased");
- freeBufferLocked(i);
- }
- }
+ if (mAbandoned) {
+ // Nothing to do if we're already abandoned.
+ return;
}
- // Clear the local buffer references. This would happen automatically
- // when the array gets dtor'd, but I'm doing it explicitly for clarity.
+ uint32_t mask = 0;
+ mBufferQueue->getReleasedBuffers(&mask);
for (int i = 0; i < BufferQueue::NUM_BUFFER_SLOTS; i++) {
- bufRefs[i].clear();
+ if (mask & (1 << i)) {
+ freeBufferLocked(i);
+ }
}
}