You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@commons.apache.org by se...@apache.org on 2009/12/18 19:44:25 UTC

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

Author: sebb
Date: Fri Dec 18 18:44:24 2009
New Revision: 892343

URL: http://svn.apache.org/viewvc?rev=892343&view=rev
Log:
LANG-568 - @SuppressWarnings("unchecked") is used too generally

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=892343&r1=892342&r2=892343&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 Fri Dec 18 18:44:24 2009
@@ -336,10 +336,10 @@
 
         private static final long serialVersionUID = 1L;
 
-        @SuppressWarnings("unchecked")
+        @SuppressWarnings("unchecked") // Comparator works for all types
         public static final ComparableComparator<?> INSTANCE = new ComparableComparator();
 
-        @SuppressWarnings("unchecked")
+        @SuppressWarnings("unchecked") // OK to cast, because comparator works for all types
         public static <E extends Comparable<? super E>> ComparableComparator<E> getInstance() {
             return (ComparableComparator<E>) INSTANCE;
         }