commit 2d3f8b8b0176d238f71148eaacbb5dfd20364a59
author Tim Murray <timmurray@google.com> Wed Dec 04 16:24:17 2019 -0800
committer Tim Murray <timmurray@google.com> Fri Jan 31 13:13:30 2020 -0800
tree a041bd6fcc53ee2ac0078e6006b9029e495f75ad
parent 19de7665668b4087cfb6cac378c4ab6df46bd107

avoid unnecessary allocation in SurfaceFlinger

Test: boots, works

Change-Id: I6371addc18dfa2aa4826432c58058cd897a190c5

services/surfaceflinger/DisplayHardware/HWComposer.cpp

diff --git a/services/surfaceflinger/DisplayHardware/HWComposer.cpp b/services/surfaceflinger/DisplayHardware/HWComposer.cpp
index 153cfe7..7124c6a 100644
--- a/services/surfaceflinger/DisplayHardware/HWComposer.cpp
+++ b/services/surfaceflinger/DisplayHardware/HWComposer.cpp
@@ -579,12 +579,13 @@
 
 sp<Fence> HWComposer::getLayerReleaseFence(DisplayId displayId, HWC2::Layer* layer) const {
     RETURN_IF_INVALID_DISPLAY(displayId, Fence::NO_FENCE);
-    auto displayFences = mDisplayData.at(displayId).releaseFences;
-    if (displayFences.count(layer) == 0) {
+    const auto& displayFences = mDisplayData.at(displayId).releaseFences;
+    auto fence = displayFences.find(layer);
+    if (fence == displayFences.end()) {
         ALOGV("getLayerReleaseFence: Release fence not found");
         return Fence::NO_FENCE;
     }
-    return displayFences[layer];
+    return fence->second;
 }
 
 status_t HWComposer::presentAndGetReleaseFences(DisplayId displayId) {