summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
author Treehugger Robot <treehugger-gerrit@google.com> 2022-03-09 18:36:49 +0000
committer Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com> 2022-03-09 18:36:49 +0000
commit995166710f9f0dc4a6f77da169756aa155341ace (patch)
tree5766d3e15512a8b1efc03a3df8edf5b13b741104
parent5e5df191f29129085cf58d7d560569c6797b6fe5 (diff)
parent83de9444cbdf1fde30b3434e4ae725b4fb8def8a (diff)
Merge "IBinder.transact: clarify return" am: 2794ff0ca8 am: 5e19eaf0e8 am: 83de9444cb
Original change: https://android-review.googlesource.com/c/platform/frameworks/base/+/1990775 Change-Id: Ia92f237101e5e930100a5723651602c7298ac0d9
-rw-r--r--core/java/android/os/IBinder.java5
1 files changed, 4 insertions, 1 deletions
diff --git a/core/java/android/os/IBinder.java b/core/java/android/os/IBinder.java
index 010459d06e8d..9e47a708162d 100644
--- a/core/java/android/os/IBinder.java
+++ b/core/java/android/os/IBinder.java
@@ -293,7 +293,10 @@ public interface IBinder {
*
* @return Returns the result from {@link Binder#onTransact}. A successful call
* generally returns true; false generally means the transaction code was not
- * understood.
+ * understood. For a oneway call to a different process false should never be
+ * returned. If a oneway call is made to code in the same process (usually to
+ * a C++ or Rust implementation), then there are no oneway semantics, and
+ * false can still be returned.
*/
public boolean transact(int code, @NonNull Parcel data, @Nullable Parcel reply, int flags)
throws RemoteException;