From ccdc71e3537609981238687f500be575112d4941 Mon Sep 17 00:00:00 2001 From: Yohei Yukawa Date: Tue, 4 Oct 2022 08:35:00 -0700 Subject: [PATCH] Simplify IInputMethodInvoker#updateEditorToolType() a bit This is a mechanical refactoring CL that simplifies IInputMethodInvoker#updateEditorToolType(), which was added recently [1], by making its return value void from boolean, because no one checks its return value. There should be no observable behavior change. [1]: I91ea9204ef62697b8942b0f350169382f18b3278 e5f499176d316d4f7e0563497e37efb585110f23 Bug: 234882948 Test: presubmit Change-Id: Id9b14d8ced97b6a57f1cc5539f4f392aa70f9e51 --- .../com/android/server/inputmethod/IInputMethodInvoker.java | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/services/core/java/com/android/server/inputmethod/IInputMethodInvoker.java b/services/core/java/com/android/server/inputmethod/IInputMethodInvoker.java index 3e397468aa875..82436cc0890a5 100644 --- a/services/core/java/com/android/server/inputmethod/IInputMethodInvoker.java +++ b/services/core/java/com/android/server/inputmethod/IInputMethodInvoker.java @@ -221,14 +221,12 @@ final class IInputMethodInvoker { } @AnyThread - boolean updateEditorToolType(int toolType) { + void updateEditorToolType(@MotionEvent.ToolType int toolType) { try { mTarget.updateEditorToolType(toolType); } catch (RemoteException e) { logRemoteException(e); - return false; } - return true; } @AnyThread