diff options
author | Endi S. Dewata <edewata@redhat.com> | 2017-06-21 22:02:47 +0200 |
---|---|---|
committer | Endi S. Dewata <edewata@redhat.com> | 2017-06-22 04:00:29 +0200 |
commit | b4c50042b821f9bca854be2fbea382738060764d (patch) | |
tree | 3490cfcfbe785b85f79dc9830fdf39415ea56b95 | |
parent | 5d0c29f2c56d7a0c55f53e93beb1fa1078a8139d (diff) | |
download | pki-b4c50042b821f9bca854be2fbea382738060764d.tar.gz pki-b4c50042b821f9bca854be2fbea382738060764d.tar.xz pki-b4c50042b821f9bca854be2fbea382738060764d.zip |
Refactored LogQueue class.
The LogQueue class has been refactored such that the list of
listeners is never null.
The Logger class has been modified to get the LogQueue singleton
directly instead of through LogSubsystem.
https://pagure.io/dogtagpki/issue/2689
Change-Id: Id5d3c132215014a238fc5ecd5f80d60d687e3dd9
3 files changed, 3 insertions, 16 deletions
diff --git a/base/server/cmscore/src/com/netscape/cmscore/logging/LogQueue.java b/base/server/cmscore/src/com/netscape/cmscore/logging/LogQueue.java index 751a4cd63..1c477f751 100644 --- a/base/server/cmscore/src/com/netscape/cmscore/logging/LogQueue.java +++ b/base/server/cmscore/src/com/netscape/cmscore/logging/LogQueue.java @@ -34,7 +34,7 @@ import com.netscape.certsrv.logging.ILogQueue; public class LogQueue implements ILogQueue { private static LogQueue mLogQueue = new LogQueue(); - protected Vector<ILogEventListener> mListeners = null; + protected Vector<ILogEventListener> mListeners = new Vector<ILogEventListener>(); /** * Constructs a log queue. @@ -52,7 +52,7 @@ public class LogQueue implements ILogQueue { * */ public void init() { - mListeners = new Vector<ILogEventListener>(); + mListeners.clear(); } @@ -61,8 +61,6 @@ public class LogQueue implements ILogQueue { * <P> */ public void shutdown() { - if (mListeners == null) - return; for (int i = 0; i < mListeners.size(); i++) { ILogEventListener listener = mListeners.elementAt(i); listener.shutdown(); @@ -96,8 +94,6 @@ public class LogQueue implements ILogQueue { * @param event the log event */ public void log(ILogEvent event) { - if (mListeners == null) - return; for (int i = 0; i < mListeners.size(); i++) { try { mListeners.elementAt(i).log(event); diff --git a/base/server/cmscore/src/com/netscape/cmscore/logging/LogSubsystem.java b/base/server/cmscore/src/com/netscape/cmscore/logging/LogSubsystem.java index aa5714668..0fe3ac622 100644 --- a/base/server/cmscore/src/com/netscape/cmscore/logging/LogSubsystem.java +++ b/base/server/cmscore/src/com/netscape/cmscore/logging/LogSubsystem.java @@ -191,13 +191,6 @@ public class LogSubsystem implements ILogSubsystem { return mInstance; } - /** - * Retrieves LogQueue. - */ - public static ILogQueue getLogQueue() { - return mLogQueue; - } - public String getLogPluginName(ILogEventListener log) { IConfigStore cs = log.getConfigStore(); if (cs == null) { diff --git a/base/server/cmscore/src/com/netscape/cmscore/logging/Logger.java b/base/server/cmscore/src/com/netscape/cmscore/logging/Logger.java index aec4caec7..c1c6e4af0 100644 --- a/base/server/cmscore/src/com/netscape/cmscore/logging/Logger.java +++ b/base/server/cmscore/src/com/netscape/cmscore/logging/Logger.java @@ -37,7 +37,7 @@ import com.netscape.certsrv.logging.ILogger; public class Logger implements ILogger { protected static Logger mLogger = new Logger(); - protected ILogQueue mLogQueue = null; + protected ILogQueue mLogQueue = LogQueue.getLogQueue(); protected Hashtable<Integer, ILogEventFactory> mFactories = new Hashtable<Integer, ILogEventFactory>(); /** @@ -45,8 +45,6 @@ public class Logger implements ILogger { * of resident event factories. */ public Logger() { - mLogQueue = LogSubsystem.getLogQueue(); - // register standard event factories register(EV_AUDIT, new AuditEventFactory()); register(EV_SYSTEM, new SystemEventFactory()); |