From: Thomas Weißschuh <thomas@t-8ch.de>
Date: Sun, 3 Jan 2021 09:26:09 +0000 (+0100)
Subject: [linuxinput] improve thread names (#9582)
X-Git-Url: https://git.basschouten.com/?a=commitdiff_plain;h=217af3704e8c3440e56546f68c39fad96a2dc903;p=openhab-addons.git

[linuxinput] improve thread names (#9582)

See #9581

Signed-off-by: Thomas Weißschuh <thomas@t-8ch.de>
---

diff --git a/bundles/org.openhab.binding.linuxinput/src/main/java/org/openhab/binding/linuxinput/internal/DeviceReadingHandler.java b/bundles/org.openhab.binding.linuxinput/src/main/java/org/openhab/binding/linuxinput/internal/DeviceReadingHandler.java
index 5f98f3f099..82bbf82e4a 100644
--- a/bundles/org.openhab.binding.linuxinput/src/main/java/org/openhab/binding/linuxinput/internal/DeviceReadingHandler.java
+++ b/bundles/org.openhab.binding.linuxinput/src/main/java/org/openhab/binding/linuxinput/internal/DeviceReadingHandler.java
@@ -63,7 +63,7 @@ public abstract class DeviceReadingHandler extends BaseThingHandler {
                             logger.warn("Could not read event", e);
                             updateStatus(ThingStatus.OFFLINE, ThingStatusDetail.COMMUNICATION_ERROR, e.getMessage());
                         }
-                    }, getClass(), getInstanceName());
+                    }, "events", thing);
                     thread.start();
                     worker = thread;
                 }
diff --git a/bundles/org.openhab.binding.linuxinput/src/main/java/org/openhab/binding/linuxinput/internal/LinuxInputDiscoveryService.java b/bundles/org.openhab.binding.linuxinput/src/main/java/org/openhab/binding/linuxinput/internal/LinuxInputDiscoveryService.java
index 0ef6f92d95..b92f56129c 100644
--- a/bundles/org.openhab.binding.linuxinput/src/main/java/org/openhab/binding/linuxinput/internal/LinuxInputDiscoveryService.java
+++ b/bundles/org.openhab.binding.linuxinput/src/main/java/org/openhab/binding/linuxinput/internal/LinuxInputDiscoveryService.java
@@ -152,7 +152,7 @@ public class LinuxInputDiscoveryService extends AbstractDiscoveryService {
                     waitForNewDevices(watcher);
                     return null;
                 });
-                Thread t = Utils.backgroundThread(job, getClass(), null);
+                Thread t = Utils.backgroundThread(job, "discovery", null);
                 t.start();
                 discoveryJob = job;
             } else {
diff --git a/bundles/org.openhab.binding.linuxinput/src/main/java/org/openhab/binding/linuxinput/internal/Utils.java b/bundles/org.openhab.binding.linuxinput/src/main/java/org/openhab/binding/linuxinput/internal/Utils.java
index 26c71d012a..2d9706ce43 100644
--- a/bundles/org.openhab.binding.linuxinput/src/main/java/org/openhab/binding/linuxinput/internal/Utils.java
+++ b/bundles/org.openhab.binding.linuxinput/src/main/java/org/openhab/binding/linuxinput/internal/Utils.java
@@ -14,6 +14,7 @@ package org.openhab.binding.linuxinput.internal;
 
 import org.eclipse.jdt.annotation.NonNullByDefault;
 import org.eclipse.jdt.annotation.Nullable;
+import org.openhab.core.thing.Thing;
 
 /**
  * Utilities
@@ -25,10 +26,10 @@ class Utils {
     private Utils() {
     }
 
-    static Thread backgroundThread(Runnable r, Class<?> clazz, @Nullable String instance) {
-        String name = LinuxInputBindingConstants.BINDING_ID + " :: " + clazz.getSimpleName();
-        if (instance != null) {
-            name += " :: " + instance;
+    static Thread backgroundThread(Runnable r, String type, @Nullable Thing thing) {
+        String name = "OH-binding-" + LinuxInputBindingConstants.BINDING_ID + "-" + type;
+        if (thing != null) {
+            name += "-" + thing.getUID();
         }
         Thread t = new Thread(r, name);
         t.setDaemon(true);