commit 6081863ce80ff231323a54e97072a17227a56820
author Steven Moreland <smoreland@google.com> Sat Feb 04 00:33:42 2017 -0800
committer Yifan Hong <elsk@google.com> Mon Feb 06 18:33:08 2017 +0000
tree 3b2e8b014dcbb63df4d753b9714d193d6f4311c0
parent 8adcb651a25c5faefb5e75166c453962a0189e1b

Add -Werror.

Also, fixed several warnings (including one bug in generateCppImpl.cpp).

Test: compiles

Bug: 32559427

Change-Id: I78375b9052c0ad447ebac638258f25bbf562712f

generateCpp.cpp

diff --git a/generateCpp.cpp b/generateCpp.cpp
index 6dcd3ca..2ea14ba 100644
--- a/generateCpp.cpp
+++ b/generateCpp.cpp
@@ -359,7 +359,6 @@
 
     if (isInterface) {
         const Interface *iface = mRootScope->getInterface();
-        const Interface *superType = iface->superType();
 
         out << "virtual bool isRemote() const ";
         if (!isIBase()) {
@@ -1444,6 +1443,9 @@
         }).endl();
         return OK;
     });
+    if (err != OK) {
+        return err;
+    }
 
     out << "::android::status_t " << klassName << "::onTransact(\n";