From be7369eeb92a2e5d791e13266d09833151a9fb44 Mon Sep 17 00:00:00 2001 From: Mathieu Baudier Date: Wed, 8 May 2024 09:00:57 +0200 Subject: [PATCH] Janitor thread per instance --- .../src/org/argeo/app/swt/ux/SwtArgeoApp.java | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/swt/org.argeo.app.swt/src/org/argeo/app/swt/ux/SwtArgeoApp.java b/swt/org.argeo.app.swt/src/org/argeo/app/swt/ux/SwtArgeoApp.java index d123aa0..851d686 100644 --- a/swt/org.argeo.app.swt/src/org/argeo/app/swt/ux/SwtArgeoApp.java +++ b/swt/org.argeo.app.swt/src/org/argeo/app/swt/ux/SwtArgeoApp.java @@ -95,7 +95,9 @@ public class SwtArgeoApp extends AbstractArgeoApp implements CmsEventSubscriber // private CmsUserManager cmsUserManager; // TODO make more optimal or via CmsSession/CmsView - private static Timer janitorTimer = new Timer(true); + // It works if static, but there is no clean way to get rid of it, so we open + // one per app + private Timer janitorTimer = new Timer(true); private Map> managedUis = new HashMap<>(); // ACR @@ -189,6 +191,8 @@ public class SwtArgeoApp extends AbstractArgeoApp implements CmsEventSubscriber if (log.isDebugEnabled()) log.info("Argeo Suite App stopped"); + // clean janitor timer so that th thread is not kept + janitorTimer.cancel(); } @Override -- 2.30.2