From 217af3704e8c3440e56546f68c39fad96a2dc903 Mon Sep 17 00:00:00 2001
From: =?utf8?q?Thomas=20Wei=C3=9Fschuh?= <thomas@t-8ch.de>
Date: Sun, 3 Jan 2021 10:26:09 +0100
Subject: [PATCH] [linuxinput] improve thread names (#9582)
MIME-Version: 1.0
Content-Type: text/plain; charset=utf8
Content-Transfer-Encoding: 8bit

See #9581

Signed-off-by: Thomas Weißschuh <thomas@t-8ch.de>
---
 .../linuxinput/internal/DeviceReadingHandler.java        | 2 +-
 .../linuxinput/internal/LinuxInputDiscoveryService.java  | 2 +-
 .../org/openhab/binding/linuxinput/internal/Utils.java   | 9 +++++----
 3 files changed, 7 insertions(+), 6 deletions(-)

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);
-- 
2.47.3