From f46fa63fdadafe5ee033faf70520bde956b02020 Mon Sep 17 00:00:00 2001
From: Hangman <webmaster@pottgames.de>
Date: Wed, 4 Jun 2025 21:43:36 +0200
Subject: [PATCH] Use ALC_SOFT_system_events in SmartDeviceRerouter

---
 .../de/pottgames/tuningfork/ALExtension.java  |  1 +
 .../router/SmartDeviceRerouter.java           | 52 ++++++++++++++++---
 2 files changed, 47 insertions(+), 6 deletions(-)

diff --git a/core/src/main/java/de/pottgames/tuningfork/ALExtension.java b/core/src/main/java/de/pottgames/tuningfork/ALExtension.java
index d49af87a..05571e6c 100644
--- a/core/src/main/java/de/pottgames/tuningfork/ALExtension.java
+++ b/core/src/main/java/de/pottgames/tuningfork/ALExtension.java
@@ -27,6 +27,7 @@ public enum ALExtension {
     ALC_SOFT_OUTPUT_MODE("ALC_SOFT_output_mode", true),
     ALC_SOFT_PAUSE_DEVICE("ALC_SOFT_pause_device", true),
     ALC_SOFT_REOPEN_DEVICE("ALC_SOFT_reopen_device", true),
+    ALC_SOFT_SYSTEM_EVENTS("ALC_SOFT_system_events", true),
     AL_EXT_ALAW("AL_EXT_ALAW", false),
     AL_EXT_BFORMAT("AL_EXT_BFORMAT", false),
     AL_EXT_DOUBLE("AL_EXT_DOUBLE", false),
diff --git a/core/src/main/java/de/pottgames/tuningfork/router/SmartDeviceRerouter.java b/core/src/main/java/de/pottgames/tuningfork/router/SmartDeviceRerouter.java
index 77782403..175fea08 100644
--- a/core/src/main/java/de/pottgames/tuningfork/router/SmartDeviceRerouter.java
+++ b/core/src/main/java/de/pottgames/tuningfork/router/SmartDeviceRerouter.java
@@ -16,6 +16,8 @@
 
 import org.lwjgl.openal.ALC10;
 import org.lwjgl.openal.EnumerateAllExt;
+import org.lwjgl.openal.SOFTSystemEventProcI;
+import org.lwjgl.openal.SOFTSystemEvents;
 import org.lwjgl.openal.SOFTReopenDevice;
 import org.lwjgl.system.MemoryUtil;
 
@@ -25,8 +27,9 @@
 import de.pottgames.tuningfork.misc.Objects;
 
 /**
- * The SmartDeviceRerouter checks every 1.5 seconds (configurable) whether the connection to the audio device still exists and whether it is the optimal
- * connection. If not, it tries to establish a connection with the following prioritization:<br>
+ * The SmartDeviceRerouter tries to keep the connection to the optimal audio device alive. When the ALC_SOFT_system_events extension is available,
+ * callbacks are used to react to device changes. Otherwise, a background thread checks every 1.5 seconds (configurable) whether the connection still
+ * exists and whether it is the optimal connection. If not, it tries to establish a connection with the following prioritization:<br>
  * <ul>
  * <li>desired device</li>
  * <li>current default device</li>
@@ -45,6 +48,8 @@ public class SmartDeviceRerouter implements AudioDeviceRerouter {
     private final long checkInterval;
 
     private volatile boolean           active                 = false;
+    private volatile boolean           useSystemEvents        = false;
+    private SOFTSystemEventProcI       systemEventProc        = null;
     private long                       device;
     private volatile ContextAttributes attributes;
     private volatile String            desiredDeviceSpecifier = null;
@@ -76,6 +81,16 @@ public void setup(long device, String desiredDeviceSpecifier, ContextAttributes
         this.attributes = attributes;
         currentDeviceSpecifier = fetchCurrentDeviceSpecifier();
         updateDesiredDevice(desiredDeviceSpecifier);
+        useSystemEvents = ALC10.alcIsExtensionPresent(device, "ALC_SOFT_system_events");
+        if (useSystemEvents) {
+            SOFTSystemEvents.alcEventControlSOFT(new int[] {
+                    SOFTSystemEvents.ALC_EVENT_TYPE_DEFAULT_DEVICE_CHANGED_SOFT,
+                    SOFTSystemEvents.ALC_EVENT_TYPE_DEVICE_ADDED_SOFT,
+                    SOFTSystemEvents.ALC_EVENT_TYPE_DEVICE_REMOVED_SOFT }, true);
+
+            systemEventProc = (eventType, deviceType, eventDevice, length, message, userParam) -> onSystemEvent(eventType);
+            SOFTSystemEvents.alcEventCallbackSOFT(systemEventProc, MemoryUtil.NULL);
+        }
         setup = true;
     }
 
@@ -103,10 +118,12 @@ public void start() {
         }
 
         active = true;
-        final Thread thread = new Thread(SmartDeviceRerouter.this::loop);
-        thread.setName("TuningFork-SmartDeviceRerouter-Thread");
-        thread.setDaemon(true);
-        thread.start();
+        if (!useSystemEvents) {
+            final Thread thread = new Thread(SmartDeviceRerouter.this::loop);
+            thread.setName("TuningFork-SmartDeviceRerouter-Thread");
+            thread.setDaemon(true);
+            thread.start();
+        }
     }
 
 
@@ -132,6 +149,22 @@ private void loop() {
         }
     }
 
+    private void onSystemEvent(int eventType) {
+        if (!active) {
+            return;
+        }
+
+        switch (eventType) {
+            case SOFTSystemEvents.ALC_EVENT_TYPE_DEFAULT_DEVICE_CHANGED_SOFT:
+            case SOFTSystemEvents.ALC_EVENT_TYPE_DEVICE_ADDED_SOFT:
+            case SOFTSystemEvents.ALC_EVENT_TYPE_DEVICE_REMOVED_SOFT:
+                tryReopen();
+                break;
+            default:
+                break;
+        }
+    }
+
 
     private synchronized void tryReopen() {
         if (desiredDeviceSpecifier == null) {
@@ -182,6 +215,13 @@ private void reopen(String deviceSpecifier) {
     @Override
     public void dispose() {
         active = false;
+        if (useSystemEvents) {
+            SOFTSystemEvents.alcEventCallbackSOFT(null, MemoryUtil.NULL);
+            SOFTSystemEvents.alcEventControlSOFT(new int[] {
+                    SOFTSystemEvents.ALC_EVENT_TYPE_DEFAULT_DEVICE_CHANGED_SOFT,
+                    SOFTSystemEvents.ALC_EVENT_TYPE_DEVICE_ADDED_SOFT,
+                    SOFTSystemEvents.ALC_EVENT_TYPE_DEVICE_REMOVED_SOFT }, false);
+        }
     }
 
 }