From 925c72b81497842c5480fea7a460f557be2acb72 Mon Sep 17 00:00:00 2001
From: Ludovic Poitou <ludovic.poitou@forgerock.com>
Date: Wed, 15 Feb 2012 23:06:31 +0000
Subject: [PATCH] Code cleanup, following FindBugs and Netbeans advices.

---
 opends/src/server/org/opends/server/core/PluginConfigManager.java |   10 ++++++++--
 1 files changed, 8 insertions(+), 2 deletions(-)

diff --git a/opends/src/server/org/opends/server/core/PluginConfigManager.java b/opends/src/server/org/opends/server/core/PluginConfigManager.java
index c4d51ab..d9516bf 100644
--- a/opends/src/server/org/opends/server/core/PluginConfigManager.java
+++ b/opends/src/server/org/opends/server/core/PluginConfigManager.java
@@ -23,7 +23,7 @@
  *
  *
  *      Copyright 2006-2010 Sun Microsystems, Inc.
- *      Portions Copyright 2011 ForgeRock AS
+ *      Portions Copyright 2011-2012 ForgeRock AS
  */
 package org.opends.server.core;
 
@@ -901,9 +901,9 @@
       DN pluginEntryDN, Set<PluginType> pluginTypes)
   {
     pluginLock.lock();
-    registeredPlugins.put(pluginEntryDN, plugin);
     try
     {
+      registeredPlugins.put(pluginEntryDN, plugin);
       registerPlugin0(plugin, pluginTypes);
     }
     finally
@@ -5459,6 +5459,7 @@
   /**
    * {@inheritDoc}
    */
+  @Override
   public boolean isConfigurationAddAcceptable(PluginCfg configuration,
                                               List<Message> unacceptableReasons)
   {
@@ -5495,6 +5496,7 @@
   /**
    * {@inheritDoc}
    */
+  @Override
   public ConfigChangeResult applyConfigurationAdd(
                                  PluginCfg configuration)
   {
@@ -5548,6 +5550,7 @@
   /**
    * {@inheritDoc}
    */
+  @Override
   public boolean isConfigurationDeleteAcceptable(
                       PluginCfg configuration,
                       List<Message> unacceptableReasons)
@@ -5561,6 +5564,7 @@
   /**
    * {@inheritDoc}
    */
+  @Override
   public ConfigChangeResult applyConfigurationDelete(
                                  PluginCfg configuration)
   {
@@ -5578,6 +5582,7 @@
   /**
    * {@inheritDoc}
    */
+  @Override
   public boolean isConfigurationChangeAcceptable(
                       PluginCfg configuration,
                       List<Message> unacceptableReasons)
@@ -5615,6 +5620,7 @@
   /**
    * {@inheritDoc}
    */
+  @Override
   public ConfigChangeResult applyConfigurationChange(
                                  PluginCfg configuration)
   {

--
Gitblit v1.10.0