From 8271ebead97bf11132eb1146c836212ae456e27c Mon Sep 17 00:00:00 2001
From: Jean-Noel Rouvignac <jean-noel.rouvignac@forgerock.com>
Date: Tue, 25 Feb 2014 13:48:08 +0000
Subject: [PATCH] Removed unnecessary use of reflection for invoking methods, aprticularly with all ConfigManagers. However, I could not do it for WorkflowElementConfigManager.

---
 opendj3-server-dev/src/server/org/opends/server/loggers/AbstractLogger.java |    4 ----
 1 files changed, 0 insertions(+), 4 deletions(-)

diff --git a/opendj3-server-dev/src/server/org/opends/server/loggers/AbstractLogger.java b/opendj3-server-dev/src/server/org/opends/server/loggers/AbstractLogger.java
index 061beb1..65e878b 100644
--- a/opendj3-server-dev/src/server/org/opends/server/loggers/AbstractLogger.java
+++ b/opendj3-server-dev/src/server/org/opends/server/loggers/AbstractLogger.java
@@ -388,9 +388,7 @@
     String className = config.getJavaClass();
     ClassPropertyDefinition pd = getJavaClassPropertyDefinition();
     try {
-      // Load the class and cast it to a LogPublisher.
       P publisher = pd.loadClass(className, logPublisherClass).newInstance();
-      // The class is valid as far as we can tell.
       return publisher.isConfigurationAcceptable(config, unacceptableReasons);
     } catch (Exception e) {
       unacceptableReasons.add(invalidLoggerClassErrorMessage.get(className, config.dn(), e));
@@ -403,10 +401,8 @@
     String className = config.getJavaClass();
     ClassPropertyDefinition pd = getJavaClassPropertyDefinition();
     try {
-      // Load the class and cast it to a LogPublisher.
       P logPublisher = pd.loadClass(className, logPublisherClass).newInstance();
       logPublisher.initializeLogPublisher(config, serverContext);
-      // The log publisher has been successfully initialized.
       return logPublisher;
     }
     catch (Exception e)

--
Gitblit v1.10.0