commit 548fdbda19d327e2285e43bf72c136938dbe6817
author Chih-Hung Hsieh <chh@google.com> Tue Dec 05 10:11:02 2017 -0800
committer Chih-Hung Hsieh <chh@google.com> Tue Dec 05 10:18:23 2017 -0800
tree 8aa413e11c7b49f026eea076ed7acdd11991dc9a
parent af807e02d07a3ee68f873830c5931f414d8df5a7

Use -Werror in frameworks/base/tools/streaming_proto

* Remove unused constants.

Bug: 66996870
Test: build with WITH_TIDY=1
Change-Id: I43940e8c446bab25b8fa841ec8c17885ab5d88ed

tools/streaming_proto/Android.bp

diff --git a/tools/streaming_proto/Android.bp b/tools/streaming_proto/Android.bp
index dc5c14e..4e9391d 100644
--- a/tools/streaming_proto/Android.bp
+++ b/tools/streaming_proto/Android.bp
@@ -24,6 +24,10 @@
         "stream_proto_utils.cpp",
         "string_utils.cpp",
     ],
+    cflags: [
+        "-Wall",
+        "-Werror",
+    ],
 
     shared_libs: ["libprotoc"],
 }

tools/streaming_proto/stream_proto_utils.cpp

diff --git a/tools/streaming_proto/stream_proto_utils.cpp b/tools/streaming_proto/stream_proto_utils.cpp
index e8f86bc..fb2d98d 100644
--- a/tools/streaming_proto/stream_proto_utils.cpp
+++ b/tools/streaming_proto/stream_proto_utils.cpp
@@ -13,8 +13,6 @@
 // TODO: packed is not supported yet.
 //
 const uint64_t FIELD_COUNT_SHIFT = 40;
-const uint64_t FIELD_COUNT_MASK = 0x0fULL << FIELD_COUNT_SHIFT;
-const uint64_t FIELD_COUNT_UNKNOWN = 0;
 const uint64_t FIELD_COUNT_SINGLE = 1ULL << FIELD_COUNT_SHIFT;
 const uint64_t FIELD_COUNT_REPEATED = 2ULL << FIELD_COUNT_SHIFT;
 const uint64_t FIELD_COUNT_PACKED = 5ULL << FIELD_COUNT_SHIFT;