commit b4ba8f5c269b6d82a09f779dc3336ece4c04c053
author Dominik Laskowski <domlaskowski@google.com> Mon Sep 27 18:20:58 2021 -0700
committer Dominik Laskowski <domlaskowski@google.com> Fri Oct 01 15:07:29 2021 -0700
tree c890be2259d2554b920439e40504cb7cd36c16b2
parent 187489657e13f5910961180af7aefb0f7c4c765a

SF: Fix -Wunused-but-set-variable

Soong appends -Wno-error=unused-but-set-variable, negating SF's -Werror.

- The type and appId variables are VR remnants.
- TimeStats::canAddNewAggregatedStats compared apples and oranges.

Bug: 129481165
Test: No warnings on clean build
Change-Id: I071e71148a269bf9ab9a5e930a775b4ae6564916

services/surfaceflinger/TimeStats/TimeStats.cpp

diff --git a/services/surfaceflinger/TimeStats/TimeStats.cpp b/services/surfaceflinger/TimeStats/TimeStats.cpp
index 7c1f21f..bf2038b 100644
--- a/services/surfaceflinger/TimeStats/TimeStats.cpp
+++ b/services/surfaceflinger/TimeStats/TimeStats.cpp
@@ -564,7 +564,7 @@
         layerRecords += record.second.stats.size();
     }
 
-    return mTimeStats.stats.size() < MAX_NUM_LAYER_STATS;
+    return layerRecords < MAX_NUM_LAYER_STATS;
 }
 
 void TimeStats::setPostTime(int32_t layerId, uint64_t frameNumber, const std::string& layerName,