From 3b67f1fbc4b781ac02f5289414db593f7aee524e Mon Sep 17 00:00:00 2001
From: Jason Macnak <natsu@google.com>
Date: Wed, 13 Dec 2023 23:19:15 +0000
Subject: Enable "-Wall" in RanchuHwc

Bug: b/316204299
Test: m
Change-Id: I38972ad064d2cad5a8d9bc9102da3989113d37d8
Merged-In: I38972ad064d2cad5a8d9bc9102da3989113d37d8
---
 system/hwc3/Android.bp            | 4 ++++
 system/hwc3/HostFrameComposer.cpp | 6 ------
 system/hwc3/VsyncThread.cpp       | 1 +
 3 files changed, 5 insertions(+), 6 deletions(-)

diff --git a/system/hwc3/Android.bp b/system/hwc3/Android.bp
index 45e122c2..94912811 100644
--- a/system/hwc3/Android.bp
+++ b/system/hwc3/Android.bp
@@ -86,6 +86,10 @@ cc_binary {
         "VsyncThread.cpp",
     ],
 
+    cflags: [
+        "-Wall",
+    ],
+
     vintf_fragments: ["hwc3.xml"],
     init_rc: ["hwc3.rc"],
 
diff --git a/system/hwc3/HostFrameComposer.cpp b/system/hwc3/HostFrameComposer.cpp
index 03739946..d9a5303c 100644
--- a/system/hwc3/HostFrameComposer.cpp
+++ b/system/hwc3/HostFrameComposer.cpp
@@ -26,8 +26,6 @@
 #include <poll.h>
 #include <sync/sync.h>
 #include <ui/GraphicBuffer.h>
-#include <ui/GraphicBufferAllocator.h>
-#include <ui/GraphicBufferMapper.h>
 
 #include <optional>
 #include <tuple>
@@ -161,10 +159,6 @@ class ComposeMsg_v2 {
   ComposeDevice_v2* mComposeDevice;
 };
 
-void FreeDisplayColorBuffer(const native_handle_t* h) {
-  ::android::GraphicBufferAllocator::get().free(h);
-}
-
 }  // namespace
 
 HWC3::Error HostFrameComposer::init() {
diff --git a/system/hwc3/VsyncThread.cpp b/system/hwc3/VsyncThread.cpp
index 350a9065..08e55b7c 100644
--- a/system/hwc3/VsyncThread.cpp
+++ b/system/hwc3/VsyncThread.cpp
@@ -172,6 +172,7 @@ void VsyncThread::threadLoop() {
       DEBUG_LOG("%s: for display:%" PRIu64 " send %" PRIu32
                 " in last %d seconds",
                 __FUNCTION__, mDisplayId, vsyncs, kLogIntervalSeconds);
+      (void)vsyncs;
       previousLog = now;
       vsyncs = 0;
     }
-- 
cgit v1.2.3