From e3f08f5681f4f08b8827b982efb0e3c42bfc9aba Mon Sep 17 00:00:00 2001
From: abobrov <abobrov@localhost>
Date: Mon, 24 Sep 2007 18:44:51 +0000
Subject: [PATCH] - fix FindBugs positives.  

---
 opends/src/server/org/opends/server/extensions/FIFOEntryCache.java |   34 +++++++++++++++++-----------------
 1 files changed, 17 insertions(+), 17 deletions(-)

diff --git a/opends/src/server/org/opends/server/extensions/FIFOEntryCache.java b/opends/src/server/org/opends/server/extensions/FIFOEntryCache.java
index 0a78dcf..63fb87e 100644
--- a/opends/src/server/org/opends/server/extensions/FIFOEntryCache.java
+++ b/opends/src/server/org/opends/server/extensions/FIFOEntryCache.java
@@ -188,24 +188,20 @@
   {
     cacheLock.lock();
 
-    registeredConfiguration.removeFIFOChangeListener (this);
+    try {
+      registeredConfiguration.removeFIFOChangeListener(this);
 
-    // Release all memory currently in use by this cache.
-    try
-    {
-      idMap.clear();
-      dnMap.clear();
-    }
-    catch (Exception e)
-    {
-      // This should never happen.
-      if (debugEnabled())
-      {
-        TRACER.debugCaught(DebugLogLevel.ERROR, e);
+      // Release all memory currently in use by this cache.
+      try {
+        idMap.clear();
+        dnMap.clear();
+      } catch (Exception e) {
+        // This should never happen.
+        if (debugEnabled()) {
+          TRACER.debugCaught(DebugLogLevel.ERROR, e);
+        }
       }
-    }
-    finally
-    {
+    } finally {
       cacheLock.unlock();
     }
   }
@@ -732,7 +728,11 @@
 
 
   /**
-   * {@inheritDoc}
+   * Clears all entries at or below the specified base DN that are associated
+   * with the given backend.  The caller must already hold the cache lock.
+   *
+   * @param  baseDN   The base DN below which all entries should be flushed.
+   * @param  backend  The backend for which to remove the appropriate entries.
    */
   private void clearSubtree(DN baseDN, Backend backend)
   {

--
Gitblit v1.10.0