Application closing bug about jworksheet

[d-led]

thanks for the pragmatic tool likewise!

[d-led]

commenting out that hook fixes the problem: https://github.com/pponec/jworksheet/blob/master/src/main/java/net/ponec/jworksheet/gui/JWorkSheet.java#L267-L273

there's already another hook, so there's probably a race condition finalizing the app

[pponec]

there's already another hook, so there's probably a race condition finalizing the app

Thank you for the notification, the bug is fixed.

[pponec]

At first glance, it looks like the application is closed properly, but unfortunately the system process will continue.

[d-led]

I've rather tried the other simple hook. This worked on OSX:

--- a/src/main/java/net/ponec/jworksheet/gui/JWorkSheet.java
+++ b/src/main/java/net/ponec/jworksheet/gui/JWorkSheet.java
@@ -253,15 +253,12 @@ public final class JWorkSheet extends TopFrame {
                 }
             });
 
-            // Set a terminate hook (https://www.baeldung.com/jvm-shutdown-hooks):
-            running = true;
-            Runtime.getRuntime().addShutdownHook(new Thread(() -> {
-                if (running) {
-                    running = false;
+            Runtime.getRuntime().addShutdownHook(new Thread(){^M
+                @Override^M
+                public void run() {^M
                     closeAppl(null);
                 }
-                System.exit(0);
-            }));

[pponec]

Thank you for information, the bug is fixed today.