On Tue, Aug 11, 2009 at 7:40 AM, <s...@apache.org> wrote:
> Author: sebb
> Date: Tue Aug 11 11:40:08 2009
> New Revision: 803067
>
> URL: http://svn.apache.org/viewvc?rev=803067&view=rev
> Log:
> Boxing/unboxing
>
> Modified:
>    
> commons/proper/jexl/branches/2.0/src/test/java/org/apache/commons/jexl/ArrayLiteralTest.java
>    
> commons/proper/jexl/branches/2.0/src/test/java/org/apache/commons/jexl/examples/ArrayTest.java
>    
> commons/proper/jexl/branches/2.0/src/test/java/org/apache/commons/jexl/examples/MethodPropertyTest.java
>
<snip/>

Don't need to do this for test classes (whats been done is fine) --
tests are easier to read with the autoboxing IMO.

-Rahul


> Modified: 
> commons/proper/jexl/branches/2.0/src/test/java/org/apache/commons/jexl/ArrayLiteralTest.java
> URL: 
> http://svn.apache.org/viewvc/commons/proper/jexl/branches/2.0/src/test/java/org/apache/commons/jexl/ArrayLiteralTest.java?rev=803067&r1=803066&r2=803067&view=diff
> ==============================================================================
> --- 
> commons/proper/jexl/branches/2.0/src/test/java/org/apache/commons/jexl/ArrayLiteralTest.java
>  (original)
> +++ 
> commons/proper/jexl/branches/2.0/src/test/java/org/apache/commons/jexl/ArrayLiteralTest.java
>  Tue Aug 11 11:40:08 2009
> @@ -47,7 +47,7 @@
>         JexlContext jc = JexlHelper.createContext();
>
>         Object o = e.evaluate( jc );
> -        Object[] check = { 5, 10 };
> +        Object[] check = { Integer.valueOf(5), Integer.valueOf(10) };
>         assertTrue( Arrays.equals(check, (Object[])o) );
>     }
>
>
> Modified: 
> commons/proper/jexl/branches/2.0/src/test/java/org/apache/commons/jexl/examples/ArrayTest.java
> URL: 
> http://svn.apache.org/viewvc/commons/proper/jexl/branches/2.0/src/test/java/org/apache/commons/jexl/examples/ArrayTest.java?rev=803067&r1=803066&r2=803067&view=diff
> ==============================================================================
> --- 
> commons/proper/jexl/branches/2.0/src/test/java/org/apache/commons/jexl/examples/ArrayTest.java
>  (original)
> +++ 
> commons/proper/jexl/branches/2.0/src/test/java/org/apache/commons/jexl/examples/ArrayTest.java
>  Tue Aug 11 11:40:08 2009
> @@ -57,7 +57,7 @@
>         e = jexl.createExpression("array[0].length()");
>         o = e.evaluate(jc);
>
> -        out.print("The length of the string at location 0 is : ", o, 21);
> +        out.print("The length of the string at location 0 is : ", o, 
> Integer.valueOf(21));
>     }
>
>     /**
>
> Modified: 
> commons/proper/jexl/branches/2.0/src/test/java/org/apache/commons/jexl/examples/MethodPropertyTest.java
> URL: 
> http://svn.apache.org/viewvc/commons/proper/jexl/branches/2.0/src/test/java/org/apache/commons/jexl/examples/MethodPropertyTest.java?rev=803067&r1=803066&r2=803067&view=diff
> ==============================================================================
> --- 
> commons/proper/jexl/branches/2.0/src/test/java/org/apache/commons/jexl/examples/MethodPropertyTest.java
>  (original)
> +++ 
> commons/proper/jexl/branches/2.0/src/test/java/org/apache/commons/jexl/examples/MethodPropertyTest.java
>  Tue Aug 11 11:40:08 2009
> @@ -71,7 +71,7 @@
>
>         e = jexl.createExpression("foo.convert(1+number)");
>         o = e.evaluate(jc);
> -        out.print("value of " + e.getExpression() + " is : ", o, 
> foo.convert(1+number));
> +        out.print("value of " + e.getExpression() + " is : ", o, 
> foo.convert(1+number.intValue()));
>
>         /*
>          * access a property
>
>
>

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org
For additional commands, e-mail: dev-h...@commons.apache.org

Reply via email to