From b30f406570114a5a7de645fd46fc7b4bc619fb09 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-config/src/main/java/org/forgerock/opendj/config/DurationPropertyDefinition.java |    4 ++--
 1 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/opendj-config/src/main/java/org/forgerock/opendj/config/DurationPropertyDefinition.java b/opendj-config/src/main/java/org/forgerock/opendj/config/DurationPropertyDefinition.java
index 0b80fb7..1ff6d1b 100644
--- a/opendj-config/src/main/java/org/forgerock/opendj/config/DurationPropertyDefinition.java
+++ b/opendj-config/src/main/java/org/forgerock/opendj/config/DurationPropertyDefinition.java
@@ -403,7 +403,7 @@
             throw PropertyException.illegalPropertyValueException(this, value);
         }
 
-        if ((upperLimit != null) && (nvalue > upperLimit)) {
+        if (upperLimit != null && nvalue > upperLimit) {
             throw PropertyException.illegalPropertyValueException(this, value);
         }
     }
@@ -451,7 +451,7 @@
 
         // Check the unit is in range - values must not be more granular
         // than the base unit.
-        if ((ms % baseUnit.getDuration()) != 0) {
+        if (ms % baseUnit.getDuration() != 0) {
             throw PropertyException.illegalPropertyValueException(this, value);
         }
 

--
Gitblit v1.10.0