You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@commons.apache.org by he...@apache.org on 2009/11/03 00:29:35 UTC

svn commit: r832192 - in /commons/proper/jexl/trunk/src/main/java/org/apache/commons/jexl: Expression.java JexlArithmetic.java

Author: henrib
Date: Mon Nov  2 23:29:35 2009
New Revision: 832192

URL: http://svn.apache.org/viewvc?rev=832192&view=rev
Log:
Javadoc fixes

Modified:
    commons/proper/jexl/trunk/src/main/java/org/apache/commons/jexl/Expression.java
    commons/proper/jexl/trunk/src/main/java/org/apache/commons/jexl/JexlArithmetic.java

Modified: commons/proper/jexl/trunk/src/main/java/org/apache/commons/jexl/Expression.java
URL: http://svn.apache.org/viewvc/commons/proper/jexl/trunk/src/main/java/org/apache/commons/jexl/Expression.java?rev=832192&r1=832191&r2=832192&view=diff
==============================================================================
--- commons/proper/jexl/trunk/src/main/java/org/apache/commons/jexl/Expression.java (original)
+++ commons/proper/jexl/trunk/src/main/java/org/apache/commons/jexl/Expression.java Mon Nov  2 23:29:35 2009
@@ -22,7 +22,7 @@
  * Represents a single JEXL expression.
  * <p>
  * This simple interface provides access to the underlying expression through
- * {@link Epxression#getExpression()}.
+ * {@link Expression#getExpression()}.
  * </p>
  *
  * <p>

Modified: commons/proper/jexl/trunk/src/main/java/org/apache/commons/jexl/JexlArithmetic.java
URL: http://svn.apache.org/viewvc/commons/proper/jexl/trunk/src/main/java/org/apache/commons/jexl/JexlArithmetic.java?rev=832192&r1=832191&r2=832192&view=diff
==============================================================================
--- commons/proper/jexl/trunk/src/main/java/org/apache/commons/jexl/JexlArithmetic.java (original)
+++ commons/proper/jexl/trunk/src/main/java/org/apache/commons/jexl/JexlArithmetic.java Mon Nov  2 23:29:35 2009
@@ -439,34 +439,30 @@
      * @param right second value
      * @return test result.
      */
+    @SuppressWarnings("unchecked")
     public boolean lessThan(Object left, Object right) {
         if ((left == right) || (left == null) || (right == null)) {
             return false;
         } else if (isFloatingPoint(left) || isFloatingPoint(right)) {
             double leftDouble = toDouble(left);
             double rightDouble = toDouble(right);
-
             return leftDouble < rightDouble;
-            } else if (left instanceof BigDecimal || right instanceof BigDecimal) {
-                BigDecimal l  = toBigDecimal(left);
-                BigDecimal r  = toBigDecimal(right);
-                return l.compareTo(r) < 0;
+        } else if (left instanceof BigDecimal || right instanceof BigDecimal) {
+            BigDecimal l  = toBigDecimal(left);
+            BigDecimal r  = toBigDecimal(right);
+            return l.compareTo(r) < 0;
         } else if (isNumberable(left) || isNumberable(right)) {
             long leftLong = toLong(left);
             long rightLong = toLong(right);
-
             return leftLong < rightLong;
         } else if (left instanceof String || right instanceof String) {
             String leftString = left.toString();
             String rightString = right.toString();
-
             return leftString.compareTo(rightString) < 0;
         } else if (left instanceof Comparable<?>) {
-            @SuppressWarnings("unchecked")
             final Comparable<Object> comparable = (Comparable<Object>) left;
             return comparable.compareTo(right) < 0;
         } else if (right instanceof Comparable<?>) {
-            @SuppressWarnings("unchecked")
             final Comparable<Object> comparable = (Comparable<Object>) right;
             return comparable.compareTo(left) > 0;
         }