From 0440760ba5af5211e7da2a120d708f11079973c4 Mon Sep 17 00:00:00 2001
From: Jean-Noel Rouvignac <jean-noel.rouvignac@forgerock.com>
Date: Fri, 22 Aug 2014 10:38:08 +0000
Subject: [PATCH] Applied AutoRefactor to com.forgerock.opendj.ldap.tools package.

---
 opendj-ldap-toolkit/src/main/java/com/forgerock/opendj/ldap/tools/LDAPCompare.java |   17 +++++------------
 1 files changed, 5 insertions(+), 12 deletions(-)

diff --git a/opendj-ldap-toolkit/src/main/java/com/forgerock/opendj/ldap/tools/LDAPCompare.java b/opendj-ldap-toolkit/src/main/java/com/forgerock/opendj/ldap/tools/LDAPCompare.java
index e83ea77..8880a39 100644
--- a/opendj-ldap-toolkit/src/main/java/com/forgerock/opendj/ldap/tools/LDAPCompare.java
+++ b/opendj-ldap-toolkit/src/main/java/com/forgerock/opendj/ldap/tools/LDAPCompare.java
@@ -97,17 +97,13 @@
       super(out, err);
     }
 
-    /**
-     * {@inheritDoc}
-     */
+    /** {@inheritDoc} */
     @Override
     public boolean isInteractive() {
         return false;
     }
 
-    /**
-     * {@inheritDoc}
-     */
+    /** {@inheritDoc} */
     @Override
     public boolean isVerbose() {
         return verbose.isPresent();
@@ -131,7 +127,7 @@
                 final Result r = ere.getResult();
                 errPrintln(ERR_TOOL_RESULT_CODE.get(r.getResultCode().intValue(), r.getResultCode()
                         .toString()));
-                if ((r.getDiagnosticMessage() != null) && (r.getDiagnosticMessage().length() > 0)) {
+                if (r.getDiagnosticMessage() != null && r.getDiagnosticMessage().length() > 0) {
                     errPrintln(LocalizableMessage.raw(r.getDiagnosticMessage()));
                 }
                 if (r.getMatchedDN() != null && r.getMatchedDN().length() > 0) {
@@ -256,14 +252,11 @@
 
         // First element should be an attribute string.
         final String attributeString = attrAndDNStrings.remove(0);
-
         // Rest are DN strings
-        for (final String s : attrAndDNStrings) {
-            dnStrings.add(s);
-        }
+        dnStrings.addAll(attrAndDNStrings);
 
         // If no DNs were provided, then exit with an error.
-        if (dnStrings.isEmpty() && (!filename.isPresent())) {
+        if (dnStrings.isEmpty() && !filename.isPresent()) {
             errPrintln(ERR_LDAPCOMPARE_NO_DNS.get());
             return ResultCode.CLIENT_SIDE_PARAM_ERROR.intValue();
         }

--
Gitblit v1.10.0