summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorChih-Hung Hsieh <chh@google.com>2017-10-02 15:50:51 -0700
committerChih-Hung Hsieh <chh@google.com>2017-10-10 14:20:21 -0700
commit8e85eb6f6ac88ab1f5a31717d3ab24fdc619546c (patch)
tree9afe4d0a6566a0fde6d450b11bfa5d6be0577916
parent6ef2b729b4e4ac070563efba29fca2b98ba09e56 (diff)
downloadTerminal-8e85eb6f6ac88ab1f5a31717d3ab24fdc619546c.tar.gz
Use -Werror in packages/apps/Terminal
* Fix unused variable warnings. Bug: 66996870 Test: build with WITH_TIDY=1 Change-Id: I0466bb537c5060d6a9a23983527fcb640eb1bd56
-rw-r--r--jni/Android.bp6
-rw-r--r--jni/com_android_terminal_Terminal.cpp1
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