From 8e85eb6f6ac88ab1f5a31717d3ab24fdc619546c Mon Sep 17 00:00:00 2001
From: Chih-Hung Hsieh <chh@google.com>
Date: Mon, 2 Oct 2017 15:50:51 -0700
Subject: Use -Werror in packages/apps/Terminal

* Fix unused variable warnings.

Bug: 66996870
Test: build with WITH_TIDY=1
Change-Id: I0466bb537c5060d6a9a23983527fcb640eb1bd56
---
 jni/Android.bp                        | 6 +++++-
 jni/com_android_terminal_Terminal.cpp | 1 -
 2 files changed, 5 insertions(+), 2 deletions(-)

diff --git a/jni/Android.bp b/jni/Android.bp
index f5ed6a7..ca8e099 100644
--- a/jni/Android.bp
+++ b/jni/Android.bp
@@ -16,5 +16,9 @@ cc_library_shared {
 
     static_libs: ["libvterm"],
 
-    cflags: ["-Wno-unused-parameter"],
+    cflags: [
+        "-Wall",
+        "-Werror",
+        "-Wno-unused-parameter",
+    ],
 }
diff --git a/jni/com_android_terminal_Terminal.cpp b/jni/com_android_terminal_Terminal.cpp
index 83aaec9..277ca89 100644
--- a/jni/com_android_terminal_Terminal.cpp
+++ b/jni/com_android_terminal_Terminal.cpp
@@ -216,7 +216,6 @@ static int term_settermprop(VTermProp prop, VTermValue *val, void *user) {
 }
 
 static int term_setmousefunc(VTermMouseFunc func, void *data, void *user) {
-    Terminal* term = reinterpret_cast<Terminal*>(user);
 #if DEBUG_CALLBACKS
     ALOGW("term_setmousefunc");
 #endif
-- 
cgit v1.2.3