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

Commit cedef056 authored by Robert Carr's avatar Robert Carr
Browse files

BLASTBufferQueue: Don't query for triple buffering 

Instead accept it as an argument so we can avoid calling
property_get_bool over and over.

Bug: 152501005
Test: Existing tests pass
Change-Id: Ie06ed7940fe883c3182950508c2f3524bf28f702
parent 9bb4cfb0

libs/gui/BLASTBufferQueue.cpp

+3 −3
Original line number Diff line number Diff line
@@ -95,7 +95,8 @@ void BLASTBufferItemConsumer::getConnectionEvents(uint64_t frameNumber, bool* ne 
    if (needsDisconnect != nullptr) *needsDisconnect = disconnect;

}



BLASTBufferQueue::BLASTBufferQueue(const sp<SurfaceControl>& surface, int width, int height)

BLASTBufferQueue::BLASTBufferQueue(const sp<SurfaceControl>& surface, int width, int height,

                                   bool enableTripleBuffering)

      : mSurfaceControl(surface),

        mWidth(width),

        mHeight(height),
@@ -105,8 +106,7 @@ BLASTBufferQueue::BLASTBufferQueue(const sp<SurfaceControl>& surface, int width, 
    // explicitly so that dequeueBuffer will block

    mProducer->setDequeueTimeout(std::numeric_limits<int64_t>::max());



    int8_t disableTripleBuffer = property_get_bool("ro.sf.disable_triple_buffer", 0);

    if (!disableTripleBuffer) {

    if (enableTripleBuffering) {

        mProducer->setMaxDequeuedBufferCount(2);

    }

    mBufferItemConsumer =

libs/gui/include/gui/BLASTBufferQueue.h

+3 −1
Original line number Diff line number Diff line
@@ -66,7 +66,9 @@ class BLASTBufferQueue 
    : public ConsumerBase::FrameAvailableListener, public BufferItemConsumer::BufferFreedListener

{

public:

    BLASTBufferQueue(const sp<SurfaceControl>& surface, int width, int height);

    BLASTBufferQueue(const sp<SurfaceControl>& surface, int width, int height,

                     bool enableTripleBuffering = true);



    sp<IGraphicBufferProducer> getIGraphicBufferProducer() const {

        return mProducer;

    }