From 455c5fbb1015c63ef8ef3be1750274c2b5773a63 Mon Sep 17 00:00:00 2001
From: Ludovic Poitou <ludovic.poitou@forgerock.com>
Date: Thu, 20 Jan 2011 18:07:33 +0000
Subject: [PATCH] Code cleanup removing potential issues detected by FindBugs
---
opends/src/server/org/opends/server/backends/jeb/VLVIndex.java | 19 +++++++++++--------
1 files changed, 11 insertions(+), 8 deletions(-)
diff --git a/opends/src/server/org/opends/server/backends/jeb/VLVIndex.java b/opends/src/server/org/opends/server/backends/jeb/VLVIndex.java
index 62117ff..444b129 100644
--- a/opends/src/server/org/opends/server/backends/jeb/VLVIndex.java
+++ b/opends/src/server/org/opends/server/backends/jeb/VLVIndex.java
@@ -23,14 +23,13 @@
*
*
* Copyright 2006-2008 Sun Microsystems, Inc.
+ * Portions Copyright 2011 ForgeRock AS
*/
package org.opends.server.backends.jeb;
import org.opends.messages.Message;
import com.sleepycat.je.*;
import org.opends.server.loggers.debug.DebugTracer;
-import static org.opends.server.loggers.debug.DebugLogger.getTracer;
-import static org.opends.server.loggers.ErrorLogger.*;
import org.opends.server.types.*;
import org.opends.server.admin.std.server.LocalDBVLVIndexCfg;
import org.opends.server.admin.server.ConfigurationChangeListener;
@@ -63,7 +62,7 @@
* This class represents a VLV index. Each database record is a sorted list
* of entry IDs followed by sets of attribute values used to sort the entries.
* The entire set of entry IDs are broken up into sorted subsets to decrease
- * the number of database retrivals needed for a range lookup. The records are
+ * the number of database retrievals needed for a range lookup. The records are
* keyed by the last entry's first sort attribute value. The list of entries
* in a particular database record maintains the property where the first sort
* attribute value is bigger then the previous key but smaller or equal
@@ -246,6 +245,7 @@
/**
* {@inheritDoc}
*/
+ @Override
public void open() throws DatabaseException
{
super.open();
@@ -278,6 +278,7 @@
* @throws DatabaseException if a JE database error occurs while
* closing the index.
*/
+ @Override
public void close() throws DatabaseException
{
super.close();
@@ -906,8 +907,8 @@
{
// Handle cases where nothing is changed early to avoid
// DB access.
- if((addedValues == null || addedValues.size() == 0) &&
- (deletedValues == null || deletedValues.size() == 0))
+ if((addedValues == null || addedValues.isEmpty()) &&
+ (deletedValues == null || deletedValues.isEmpty()))
{
return;
}
@@ -1870,9 +1871,11 @@
resultCode = ResultCode.INVALID_ATTRIBUTE_SYNTAX;
}
}
- // BUG: attrType may be NULL
- sortKeys[i] = new SortKey(attrType, ascending[i]);
- orderingRules[i] = attrType.getOrderingMatchingRule();
+ else
+ {
+ sortKeys[i] = new SortKey(attrType, ascending[i]);
+ orderingRules[i] = attrType.getOrderingMatchingRule();
+ }
}
this.sortOrder = new SortOrder(sortKeys);
--
Gitblit v1.10.0