Use the thread pool for things so its shutdown properly

Fixes #53
MCXboxBroadcast · Jun 19, 2024 · 84d2052 · 84d2052
1 parent 7013745
commit 84d2052
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 5 deletions.
diff --git a/...r/src/main/java/com/rtm516/mcxboxbroadcast/bootstrap/geyser/MCXboxBroadcastExtension.java b/...r/src/main/java/com/rtm516/mcxboxbroadcast/bootstrap/geyser/MCXboxBroadcastExtension.java
@@ -100,7 +100,7 @@ private void restart() {
         sessionManager = new SessionManager(this.dataFolder().toString(), logger);
 
         // Pull onto another thread so we don't hang the main thread
-        new Thread(this::createSession);
+        sessionManager.scheduledThread().execute(this::createSession);
     }
 
     @Subscribe
@@ -112,7 +112,7 @@ public void onPostInitialize(GeyserPostInitializeEvent event) {
         config = ConfigLoader.load(this, MCXboxBroadcastExtension.class, ExtensionConfig.class);
 
         // Pull onto another thread so we don't hang the main thread
-        new Thread(() -> {
+        sessionManager.scheduledThread().execute(() -> {
             // Get the ip to broadcast
             String ip = config.remoteAddress();
             if (ip.equals("auto")) {
@@ -152,7 +152,7 @@ public void onPostInitialize(GeyserPostInitializeEvent event) {
             sessionInfo.setPort(port);
 
             createSession();
-        }).start();
+        });
     }
 
     private void createSession() {

diff --git a/core/src/main/java/com/rtm516/mcxboxbroadcast/core/SessionManager.java b/core/src/main/java/com/rtm516/mcxboxbroadcast/core/SessionManager.java
@@ -94,7 +94,7 @@ public void init(SessionInfo sessionInfo, FriendSyncConfig friendSyncConfig) thr
 
         // Create the sub-sessions in a new thread so we don't block the main thread
         List<String> finalSubSessions = subSessions;
-        new Thread(() -> {
+        scheduledThreadPool.execute(() -> {
             // Create the sub-session manager for each sub-session
             for (String subSession : finalSubSessions) {
                 try {
@@ -107,7 +107,7 @@ public void init(SessionInfo sessionInfo, FriendSyncConfig friendSyncConfig) thr
                     // TODO Retry creation after 30s or so
                 }
             }
-        }).start();
+        });
     }
 
     @Override