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

Commit b7ebad6e authored by Alec Mouri's avatar Alec Mouri
Browse files

[AChoreographer] Add missing vector::erase call 

Otherwise we don't reduce the physical size of mRefreshRateCallbacks.

Bug: 136262896
Test: builds
Change-Id: I442a3fd7d41460f9e7617294df88cf7fb1efeab8
parent d5e21492

libs/nativedisplay/AChoreographer.cpp

+5 −4
Original line number Diff line number Diff line
@@ -154,10 +154,11 @@ void Choreographer::registerRefreshRateCallback(AChoreographer_refreshRateCallba 
void Choreographer::unregisterRefreshRateCallback(AChoreographer_refreshRateCallback cb) {

    {

        AutoMutex _l{mLock};

        std::remove_if(mRefreshRateCallbacks.begin(), mRefreshRateCallbacks.end(),

        mRefreshRateCallbacks.erase(std::remove_if(mRefreshRateCallbacks.begin(),

                                                   mRefreshRateCallbacks.end(),

                                                   [&](const RefreshRateCallback& callback) {

                                                       return cb == callback.callback;

                       });

                                                   }));

        if (mRefreshRateCallbacks.empty()) {

            toggleConfigEvents(ISurfaceComposer::ConfigChanged::eConfigChangedSuppress);

        }