You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@commons.apache.org by ba...@apache.org on 2009/12/17 12:13:39 UTC

svn commit: r891635 - /commons/proper/lang/trunk/src/java/org/apache/commons/lang3/Range.java

Author: bayard
Date: Thu Dec 17 11:13:39 2009
New Revision: 891635

URL: http://svn.apache.org/viewvc?rev=891635&view=rev
Log:
Improving doc to indicate null->false for elementBefore(T) and elementAfter(T). LANG-551

Modified:
    commons/proper/lang/trunk/src/java/org/apache/commons/lang3/Range.java

Modified: commons/proper/lang/trunk/src/java/org/apache/commons/lang3/Range.java
URL: http://svn.apache.org/viewvc/commons/proper/lang/trunk/src/java/org/apache/commons/lang3/Range.java?rev=891635&r1=891634&r2=891635&view=diff
==============================================================================
--- commons/proper/lang/trunk/src/java/org/apache/commons/lang3/Range.java (original)
+++ commons/proper/lang/trunk/src/java/org/apache/commons/lang3/Range.java Thu Dec 17 11:13:39 2009
@@ -180,12 +180,14 @@
     /**
      * <p>Tests whether the specified element occurs before this range.</p>
      *
-     * @param element  the element to test
+     * <p><code>null</code> is handled and returns <code>false</code>.</p>
+     *
+     * @param element  the element to test, may be <code>null</code>
      * @return <code>true</code> if the specified element occurs before this range
      */
     public boolean elementBefore(T element) {
         if (element == null) {
-            return false; // ??
+            return false;
         }
         
         return this.comparator.compare(element, getMinimum()) < 0;
@@ -194,12 +196,14 @@
     /**
      * <p>Tests whether the specified element occurs after this range.</p>
      *
-     * @param element  the element to test
+     * <p><code>null</code> is handled and returns <code>false</code>.</p>
+     *
+     * @param element  the element to test, may be <code>null</code>
      * @return <code>true</code> if the specified element occurs after this range
      */
     public boolean elementAfter(T element) {
         if (element == null) {
-            return false; // ??
+            return false;
         }
         
         return this.comparator.compare(element, getMaximum()) > 0;