From 3f81ea6588e42e1a2e35d020868bd42184fa3cf2 Mon Sep 17 00:00:00 2001 From: Maik Musall Date: Thu, 7 Apr 2016 11:38:48 +0200 Subject: [PATCH] Encourage named threads for easier debugging --- .../Sources/er/extensions/appserver/ERXShutdownHook.java | 9 ++++++--- 1 file changed, 6 insertions(+), 3 deletions(-) diff --git a/Frameworks/Core/ERExtensions/Sources/er/extensions/appserver/ERXShutdownHook.java b/Frameworks/Core/ERExtensions/Sources/er/extensions/appserver/ERXShutdownHook.java index 2c28971453c..f38759b0069 100644 --- a/Frameworks/Core/ERExtensions/Sources/er/extensions/appserver/ERXShutdownHook.java +++ b/Frameworks/Core/ERExtensions/Sources/er/extensions/appserver/ERXShutdownHook.java @@ -53,7 +53,7 @@ public abstract class ERXShutdownHook extends Thread { public static void initERXShutdownHook() { System.out.println( "WILL ADD SHUTDOWNHOOK" ); - Runtime.getRuntime().addShutdownHook( new Thread() { + Runtime.getRuntime().addShutdownHook( new Thread( "shutdown_complete_message_writer" ) { @Override public void run() { try { @@ -90,7 +90,8 @@ public static void useMe() { } /** - * Construct a new nameless shutdown hook and register it. + * Construct a new nameless shutdown hook and register it. It is recommended to use named hooks wherever + * possible for easier debugging. */ public ERXShutdownHook() { Runtime.getRuntime().addShutdownHook( this ); @@ -102,8 +103,10 @@ public ERXShutdownHook() { * @param hookName hook name */ public ERXShutdownHook( String hookName ) { - this(); + super( hookName ); name = hookName; + Runtime.getRuntime().addShutdownHook( this ); + ALL_HOOKS.add( this ); } @Override