commit 08d7c813f8944d967e6ca32a3f1ab5d2643ac252
author Yi Kong <yikong@google.com> Tue Dec 12 16:40:22 2023 +0900
committer Yi Kong <yikong@google.com> Wed Dec 13 15:47:51 2023 +0900
tree e8d13874af2cd3a7e1d47d12541d9cb4617ad733
parent edf5d0fd649fbfbe3ae681b8bcfabb56fea2c5f5

surfaceflinger: Fix -Wunused-variable compiler warning

Test: presubmit
Change-Id: I72a7d888cbaee4dd782ec43c1fc130669d9eb2ac

services/surfaceflinger/Scheduler/Scheduler.cpp

diff --git a/services/surfaceflinger/Scheduler/Scheduler.cpp b/services/surfaceflinger/Scheduler/Scheduler.cpp
index 68927bd..54ff2cd 100644
--- a/services/surfaceflinger/Scheduler/Scheduler.cpp
+++ b/services/surfaceflinger/Scheduler/Scheduler.cpp
@@ -577,7 +577,8 @@
     if (!scheduleOpt) return;
     const auto& schedule = scheduleOpt->get();
 
-    if (const bool needMoreSignals = schedule->getController().addPresentFence(std::move(fence))) {
+    const bool needMoreSignals = schedule->getController().addPresentFence(std::move(fence));
+    if (needMoreSignals) {
         schedule->enableHardwareVsync();
     } else {
         constexpr bool kDisallow = false;