commit f18615b57df4f164d464a23e7921c7a3b6f5e6c7
author Steven Moreland <smoreland@google.com> Fri Sep 14 11:43:06 2018 -0700
committer Steven Moreland <smoreland@google.com> Fri Sep 14 12:54:39 2018 -0700
tree 6a030d0f11a09f79b992d97d24d05d6dc18369c6
parent 5d62e445c33f0e14b8ae7d17c49f40a9b2b4221a

libbinder_ndk: AParcel 'operator->' -> 'get'

Using 'operator->' directly is not very readable, and I'd rather not
have both. This is much cleaner.

Bug: 111445392
Test: ./ndk/runtests.sh
Change-Id: Ifcd828ee226225ddd298719e640f0d22bd9e5013

libs/binder/ndk/ibinder.cpp

diff --git a/libs/binder/ndk/ibinder.cpp b/libs/binder/ndk/ibinder.cpp
index 23d84c2..b1b4194 100644
--- a/libs/binder/ndk/ibinder.cpp
+++ b/libs/binder/ndk/ibinder.cpp
@@ -426,7 +426,7 @@
     }
 
     *in = new AParcel(binder);
-    status_t status = (**in)->writeInterfaceToken(clazz->getInterfaceDescriptor());
+    status_t status = (*in)->get()->writeInterfaceToken(clazz->getInterfaceDescriptor());
     binder_status_t ret = PruneStatusT(status);
 
     if (ret != STATUS_OK) {
@@ -472,8 +472,7 @@
 
     *out = new AParcel(binder);
 
-    status_t status =
-            binder->getBinder()->transact(code, *(*in)->operator->(), (*out)->operator->(), flags);
+    status_t status = binder->getBinder()->transact(code, *(*in)->get(), (*out)->get(), flags);
     binder_status_t ret = PruneStatusT(status);
 
     if (ret != STATUS_OK) {