From faa3a7ca59efe6e4747087ee94818e6f2a29294e Mon Sep 17 00:00:00 2001 From: Werner Dittmann Date: Thu, 3 Jun 2010 13:41:54 +0000 Subject: Fix logger usage - fifth bunch. --- .../impl/notification/LogMessageNotificationHandlerImpl.java | 6 ++++-- .../communicator/impl/notification/NotificationActivator.java | 9 ++++++--- .../communicator/impl/notification/NotificationServiceImpl.java | 9 ++++++--- 3 files changed, 16 insertions(+), 8 deletions(-) (limited to 'src/net/java/sip/communicator/impl/notification') diff --git a/src/net/java/sip/communicator/impl/notification/LogMessageNotificationHandlerImpl.java b/src/net/java/sip/communicator/impl/notification/LogMessageNotificationHandlerImpl.java index 2aef8b8..a10a632 100644 --- a/src/net/java/sip/communicator/impl/notification/LogMessageNotificationHandlerImpl.java +++ b/src/net/java/sip/communicator/impl/notification/LogMessageNotificationHandlerImpl.java @@ -58,9 +58,11 @@ public class LogMessageNotificationHandlerImpl if (logType.equals(LogMessageNotificationHandler.ERROR_LOG_TYPE)) logger.error(message); else if(logType.equals(LogMessageNotificationHandler.INFO_LOG_TYPE)) - logger.info(message); + if (logger.isInfoEnabled()) + logger.info(message); else if(logType.equals(LogMessageNotificationHandler.TRACE_LOG_TYPE)) - logger.trace(message); + if (logger.isTraceEnabled()) + logger.trace(message); } /** diff --git a/src/net/java/sip/communicator/impl/notification/NotificationActivator.java b/src/net/java/sip/communicator/impl/notification/NotificationActivator.java index 588f668..b958146 100644 --- a/src/net/java/sip/communicator/impl/notification/NotificationActivator.java +++ b/src/net/java/sip/communicator/impl/notification/NotificationActivator.java @@ -42,12 +42,14 @@ public class NotificationActivator NotificationService notificationService = new NotificationServiceImpl(); - logger.info("Notification Service...[ STARTED ]"); + if (logger.isInfoEnabled()) + logger.info("Notification Service...[ STARTED ]"); bundleContext.registerService(NotificationService.class.getName(), notificationService, null); - logger.info("Notification Service ...[REGISTERED]"); + if (logger.isInfoEnabled()) + logger.info("Notification Service ...[REGISTERED]"); logger.logEntry(); } @@ -58,7 +60,8 @@ public class NotificationActivator public void stop(BundleContext bc) throws Exception { - logger.info("UI Service ...[STOPPED]"); + if (logger.isInfoEnabled()) + logger.info("UI Service ...[STOPPED]"); } /** diff --git a/src/net/java/sip/communicator/impl/notification/NotificationServiceImpl.java b/src/net/java/sip/communicator/impl/notification/NotificationServiceImpl.java index 7621940..7932018 100644 --- a/src/net/java/sip/communicator/impl/notification/NotificationServiceImpl.java +++ b/src/net/java/sip/communicator/impl/notification/NotificationServiceImpl.java @@ -182,7 +182,8 @@ public class NotificationServiceImpl String actionDescriptor, String defaultMessage) { - logger.debug("Registering event " + eventType + "/" + + if (logger.isDebugEnabled()) + logger.debug("Registering event " + eventType + "/" + actionType + "/" + actionDescriptor + "/" + defaultMessage); if (actionType.equals(NotificationService.ACTION_SOUND)) @@ -756,7 +757,8 @@ public class NotificationServiceImpl private void fireNotificationEventTypeEvent(String eventType, String sourceEventType) { - logger.debug("Dispatching NotificationEventType Change. Listeners=" + if (logger.isDebugEnabled()) + logger.debug("Dispatching NotificationEventType Change. Listeners=" + changeListeners.size() + " evt=" + eventType); @@ -956,7 +958,8 @@ public class NotificationServiceImpl String actionDescriptor, String defaultMessage) { - logger.debug("Registering default event " + eventType + "/" + + if (logger.isDebugEnabled()) + logger.debug("Registering default event " + eventType + "/" + actionType + "/" + actionDescriptor + "/" + defaultMessage); if(isDefault(eventType, actionType)) -- cgit v1.1