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

Commit 754fcb8c authored by Greg Kaiser's avatar Greg Kaiser
Browse files

Avoid multiple map lookups 

We directly use erase(), and its return value, so we only need
to look up 'deviceName' once.

Test: TreeHugger
Bug: 191248460
Change-Id: I7c229e7d1de5f76071f55f965eab447b6c868bcc
parent 337cf74c

camera/provider/2.7/default/ExternalCameraProviderImpl_2_7.cpp

+2 −3
Original line number Diff line number Diff line
@@ -292,8 +292,7 @@ void ExternalCameraProviderImpl_2_7::deviceRemoved(const char* devName) { 
    } else {

        deviceName = std::string("device@3.4/external/") + cameraId;

    }

    if (mCameraStatusMap.find(deviceName) != mCameraStatusMap.end()) {

        mCameraStatusMap.erase(deviceName);

    if (mCameraStatusMap.erase(deviceName) != 0) {

        if (mCallbacks != nullptr) {

            mCallbacks->cameraDeviceStatusChange(deviceName, CameraDeviceStatus::NOT_PRESENT);

        }