Warning and codeanalysis (openhab#15828)

Signed-off-by: Leo Siepel <[email protected]>
jlaur · Nov 3, 2023 · fddacb4 · fddacb4
1 parent 7724f16
commit fddacb4
Show file tree

Hide file tree

Showing 3 changed files with 7 additions and 8 deletions.
diff --git a/...openhab.binding.bluetooth/src/main/java/org/openhab/binding/bluetooth/BluetoothUtils.java b/...openhab.binding.bluetooth/src/main/java/org/openhab/binding/bluetooth/BluetoothUtils.java
@@ -28,7 +28,7 @@
 @NonNullByDefault
 public class BluetoothUtils {
 
-    public static final Logger logger = LoggerFactory.getLogger(BluetoothUtils.class);
+    public static final Logger LOGGER = LoggerFactory.getLogger(BluetoothUtils.class);
 
     public static final int FORMAT_UINT8 = 0x11;
     public static final int FORMAT_UINT16 = 0x12;
@@ -47,9 +47,7 @@ public class BluetoothUtils {
      */
     public static int[] toIntArray(byte[] value) {
         int[] ret = new int[value.length];
-        for (int i = 0; i < value.length; i++) {
-            ret[i] = value[i];
-        }
+        System.arraycopy(value, 0, ret, 0, value.length);
         return ret;
     }
 
@@ -90,7 +88,7 @@ public static byte[] toByteArray(int[] value) {
                 return unsignedToSigned(
                         unsignedBytesToInt(value[offset], value[offset + 1], value[offset + 2], value[offset + 3]), 32);
             default:
-                logger.error("Unknown format type {} - no int value can be provided for it.", formatType);
+                LOGGER.error("Unknown format type {} - no int value can be provided for it.", formatType);
         }
 
         return null;
@@ -111,7 +109,7 @@ public static byte[] toByteArray(int[] value) {
             case FORMAT_FLOAT:
                 return bytesToFloat(value[offset], value[offset + 1], value[offset + 2], value[offset + 3]);
             default:
-                logger.error("Unknown format type {} - no float value can be provided for it.", formatType);
+                LOGGER.error("Unknown format type {} - no float value can be provided for it.", formatType);
         }
 
         return null;

diff --git a/...c/main/java/org/openhab/binding/bluetooth/discovery/internal/BluetoothDeviceSnapshot.java b/...c/main/java/org/openhab/binding/bluetooth/discovery/internal/BluetoothDeviceSnapshot.java
@@ -165,7 +165,6 @@ public int hashCode() {
         return result;
     }
 
-    @SuppressWarnings("PMD.SimplifyBooleanReturns")
     @Override
     public boolean equals(@Nullable Object obj) {
         if (this == obj) {

diff --git a/...main/java/org/openhab/binding/bluetooth/discovery/internal/BluetoothDiscoveryService.java b/...main/java/org/openhab/binding/bluetooth/discovery/internal/BluetoothDiscoveryService.java
@@ -153,7 +153,9 @@ public void deviceDiscovered(BluetoothDevice device) {
         logger.debug("Discovered bluetooth device '{}': {}", device.getName(), device);
 
         DiscoveryCache cache = discoveryCaches.computeIfAbsent(device.getAddress(), addr -> new DiscoveryCache());
-        cache.handleDiscovery(device);
+        if (cache != null) {
+            cache.handleDiscovery(device);
+        }
     }
 
     private static ThingUID createThingUIDWithBridge(DiscoveryResult result, BluetoothAdapter adapter) {