commit e7ff228b0fd676e9a535f209276d451344850f81
author Andreas Huber <andih@google.com> Tue Aug 16 13:50:03 2016 -0700
committer Andreas Huber <andih@google.com> Tue Aug 16 13:50:03 2016 -0700
tree afe4dcea787023f01a99a5a18ea41998bc35da75
parent c7dfef3fda2061dcaeeb144c10eef3cf7360b03c

Introduce locals to be read from a parcel ahead of time

so that error handling no longer crosses variable declarations.

Bug: 30570468
Change-Id: I365fec3cc33a8fdcd03a84bcbe05848ba290c475

Interface.cpp

diff --git a/Interface.cpp b/Interface.cpp
index 42186af..2c65865 100644
--- a/Interface.cpp
+++ b/Interface.cpp
@@ -62,6 +62,9 @@
         parcelObj + (parcelObjIsPointer ? "->" : ".");
 
     if (isReader) {
+        out << "{\n";
+        out.indent();
+
         const std::string binderName = "_hidl_" + name + "_binder";
 
         out << "::android::sp<::android::hardware::IBinder> "
@@ -81,6 +84,9 @@
             << "::asInterface("
             << binderName
             << ");\n";
+
+        out.unindent();
+        out << "}\n\n";
     } else {
         out << "_hidl_err = ";
         out << parcelObjDeref