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
diff --git a/camera/provider/2.7/default/ExternalCameraProviderImpl_2_7.cpp b/camera/provider/2.7/default/ExternalCameraProviderImpl_2_7.cpp
index c812d54..b63e3bb 100644
--- a/camera/provider/2.7/default/ExternalCameraProviderImpl_2_7.cpp
+++ b/camera/provider/2.7/default/ExternalCameraProviderImpl_2_7.cpp
@@ -292,8 +292,7 @@
} 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);
}
@@ -391,4 +390,4 @@
} // namespace provider
} // namespace camera
} // namespace hardware
-} // namespace android
\ No newline at end of file
+} // namespace android