commit 426777c867a1da5c9271cfcc360a9a3154b5db56
author Michael W <baddaemon87@gmail.com> Sun Jun 25 18:38:00 2023 +0200
committer micky387 <mickaelsaibi@free.fr> Wed Jan 15 15:48:34 2025 -0500
tree 0dde404fc4b0bc5442a9c39b555777ca92005489
parent cea4252e8924823cb552128e6eb2aa5bd2912ace

ChargingControl: No need to log a crash when not supported

Instead of relying on catching the exception later on, check if
ChargingControl is actually supported before checking if a specific
mode is

Change-Id: Icaf8d333842c9847bea00bedb4b302b7acd49bb3

src/org/omnirom/omnilib/internal/health/ChargingControlController.java

diff --git a/src/org/omnirom/omnilib/internal/health/ChargingControlController.java b/src/org/omnirom/omnilib/internal/health/ChargingControlController.java
index f3237f2..a278802 100644
--- a/src/org/omnirom/omnilib/internal/health/ChargingControlController.java
+++ b/src/org/omnirom/omnilib/internal/health/ChargingControlController.java
@@ -255,7 +255,7 @@
 
     public boolean isChargingModeSupported(int mode) {
         try {
-            return (mChargingControl.getSupportedMode() & mode) != 0;
+            return isSupported() && (mChargingControl.getSupportedMode() & mode) != 0;
         } catch (RemoteException e) {
             throw new RuntimeException(e);
         }