diff options
| author | Chih-Hung Hsieh <chh@google.com> | 2017-10-03 09:57:55 -0700 |
|---|---|---|
| committer | Chih-Hung Hsieh <chh@google.com> | 2017-10-13 16:25:16 -0700 |
| commit | c7edf078f92c7ce083f8c243a79f8aecdfff4ac1 (patch) | |
| tree | d7816094d61913df8ee8c17e60763ccd050c4631 /tools/incident_report | |
| parent | 26817938f6966928488e92de73359c3887006642 (diff) | |
Use -Werror in frameworks/base
* Fix unused variable and return value warnings.
Bug: 66996870
Test: build with WITH_TIDY=1
Change-Id: I890e65a20848d00559ba5a4f9691be1347b456af
Diffstat (limited to 'tools/incident_report')
| -rw-r--r-- | tools/incident_report/Android.bp | 2 | ||||
| -rw-r--r-- | tools/incident_report/printer.cpp | 1 |
2 files changed, 1 insertions, 2 deletions
diff --git a/tools/incident_report/Android.bp b/tools/incident_report/Android.bp index ab55dbd81821..f2d0d0f3e553 100644 --- a/tools/incident_report/Android.bp +++ b/tools/incident_report/Android.bp @@ -31,5 +31,5 @@ cc_binary_host { "libprotobuf-cpp-full", ], - cflags: ["-Wno-unused-parameter"], + cflags: ["-Wall", "-Werror"], } diff --git a/tools/incident_report/printer.cpp b/tools/incident_report/printer.cpp index bd660dd20dfd..bff1025ad8da 100644 --- a/tools/incident_report/printer.cpp +++ b/tools/incident_report/printer.cpp @@ -70,7 +70,6 @@ Out::printf(const char* format, ...) len = vsnprintf(mBuf, mBufSize, format, args); va_end(args); - bool truncated = (len >= mBufSize) && (reallocate(len) < len); va_start(args, format); len = vsnprintf(mBuf, mBufSize, format, args); |