From 7d7913228e6f52273f2fdada6822a380f95e0f8c Mon Sep 17 00:00:00 2001
From: Jean-Noel Rouvignac <jean-noel.rouvignac@forgerock.com>
Date: Mon, 20 Oct 2014 12:11:48 +0000
Subject: [PATCH] Autorefactored simplifying expressions: - removed useless parentheses - added parentheses to disambiguate expressions - removed useless use of "this" keyword in method calls - removed useless null checks in conjunction with use of instanceof operator

---
 opendj-sdk/opendj-cli/src/main/java/com/forgerock/opendj/cli/ArgumentGroup.java |    4 ++--
 1 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/opendj-sdk/opendj-cli/src/main/java/com/forgerock/opendj/cli/ArgumentGroup.java b/opendj-sdk/opendj-cli/src/main/java/com/forgerock/opendj/cli/ArgumentGroup.java
index 3c0101b..2ac104e 100644
--- a/opendj-sdk/opendj-cli/src/main/java/com/forgerock/opendj/cli/ArgumentGroup.java
+++ b/opendj-sdk/opendj-cli/src/main/java/com/forgerock/opendj/cli/ArgumentGroup.java
@@ -87,8 +87,8 @@
             // new argument should replace
             for (final Iterator<Argument> it = this.args.iterator(); it.hasNext();) {
                 final Argument a = it.next();
-                if (newShort != null && newShort.equals(a.getShortIdentifier()) || newLong != null
-                        && newLong.equals(a.getLongIdentifier())) {
+                if ((newShort != null && newShort.equals(a.getShortIdentifier()))
+                        || (newLong != null && newLong.equals(a.getLongIdentifier()))) {
                     it.remove();
                     break;
                 }

--
Gitblit v1.10.0