From 898165ce0bed7698c35006cf87cd123d4bd978f0 Mon Sep 17 00:00:00 2001 From: cpfeiffer Date: Sat, 15 Sep 2018 23:30:39 +0200 Subject: [PATCH] Avoid lots of false static analysis warnings --- .../service/DeviceCommunicationService.java | 15 +++++++++++++-- 1 file changed, 13 insertions(+), 2 deletions(-) diff --git a/app/src/main/java/nodomain/freeyourgadget/gadgetbridge/service/DeviceCommunicationService.java b/app/src/main/java/nodomain/freeyourgadget/gadgetbridge/service/DeviceCommunicationService.java index b96b663c1..2929fdda7 100644 --- a/app/src/main/java/nodomain/freeyourgadget/gadgetbridge/service/DeviceCommunicationService.java +++ b/app/src/main/java/nodomain/freeyourgadget/gadgetbridge/service/DeviceCommunicationService.java @@ -339,6 +339,19 @@ public class DeviceCommunicationService extends Service implements SharedPrefere mGBDevice.sendDeviceUpdateIntent(this); } break; + default: + if (mDeviceSupport == null || mGBDevice == null) { + LOG.warn("device support:" + mDeviceSupport + ", device: " + mGBDevice + ", aborting"); + } else { + handleAction(intent, action, prefs); + } + break; + } + return START_STICKY; + } + + private void handleAction(Intent intent, String action, Prefs prefs) { + switch (action) { case ACTION_REQUEST_DEVICEINFO: mGBDevice.sendDeviceUpdateIntent(this); break; @@ -567,8 +580,6 @@ public class DeviceCommunicationService extends Service implements SharedPrefere } break; } - - return START_STICKY; } /**