commit 90cde57df677a9d81cee55954d1393c783e94382
author Bernie Innocenti <codewiz@google.com> Tue Oct 08 15:41:57 2019 +0900
committer Bernie Innocenti <codewiz@google.com> Tue Oct 08 15:44:04 2019 +0900
tree efafda2ff0ce9d5570e9bcd2656a44857a5eb3c2
parent 7c5d43915046b2d1af8930ae772c291ea28baac3

Enable stricter compiler warnings

Test: m apf_run apf_disassembler libapf
Change-Id: I43b975a6afa11f693268317de69967e7774a1c85

Android.bp

diff --git a/Android.bp b/Android.bp
index 6832407..d2185b5 100644
--- a/Android.bp
+++ b/Android.bp
@@ -7,6 +7,12 @@
         "-DAPF_FRAME_HEADER_SIZE=14",
         "-Wall",
         "-Werror",
+        "-Werror=implicit-fallthrough",
+        "-Wnullable-to-nonnull-conversion",
+        "-Wsign-compare",
+        "-Wthread-safety",
+        "-Wunused-parameter",
+        "-Wuninitialized",
     ],
 }
 

apf_run.c

diff --git a/apf_run.c b/apf_run.c
index 3f89b13..c915b80 100644
--- a/apf_run.c
+++ b/apf_run.c
@@ -178,7 +178,7 @@
     char* packet = NULL;
     uint8_t* data = NULL;
     uint32_t data_len = 0;
-    int32_t filter_age = 0;
+    uint32_t filter_age = 0;
 
     int opt;
     char *endptr;