commit d66be0ae6929fee064a229d6c6bac3a871aba137
author Chih-Hung Hsieh <chh@google.com> Thu Oct 05 13:51:32 2017 -0700
committer Chih-Hung Hsieh <chh@google.com> Fri Oct 13 16:12:08 2017 -0700
tree edf6df30381838963d504cc4e0e68a7ec8f515b9
parent afee092bee459332c8d577163794581c3128df4f

Use -Werror in frameworks/native/libs/gui and ui

* Remove unused variable and function.
* Suppress warning of unused return results.

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

libs/gui/Surface.cpp

diff --git a/libs/gui/Surface.cpp b/libs/gui/Surface.cpp
index 7b2b5c3..5bee7a4 100644
--- a/libs/gui/Surface.cpp
+++ b/libs/gui/Surface.cpp
@@ -153,7 +153,10 @@
     ATRACE_CALL();
 
     DisplayStatInfo stats;
-    status_t err = composerService()->getDisplayStats(NULL, &stats);
+    status_t result = composerService()->getDisplayStats(NULL, &stats);
+    if (result != NO_ERROR) {
+        return result;
+    }
 
     *outRefreshDuration = stats.vsyncPeriod;