On 28 May 2012 16:39,  <celes...@apache.org> wrote:
> Author: celestin
> Date: Mon May 28 15:39:57 2012
> New Revision: 1343293
>
> URL: http://svn.apache.org/viewvc?rev=1343293&view=rev
> Log:
> Removed explicit conversion from int to Integer.

-1; I don't think we had concensus on removing it.

>
> Modified:
>    
> commons/proper/math/trunk/src/main/java/org/apache/commons/math3/linear/RealVector.java
>
> Modified: 
> commons/proper/math/trunk/src/main/java/org/apache/commons/math3/linear/RealVector.java
> URL: 
> http://svn.apache.org/viewvc/commons/proper/math/trunk/src/main/java/org/apache/commons/math3/linear/RealVector.java?rev=1343293&r1=1343292&r2=1343293&view=diff
> ==============================================================================
> --- 
> commons/proper/math/trunk/src/main/java/org/apache/commons/math3/linear/RealVector.java
>  (original)
> +++ 
> commons/proper/math/trunk/src/main/java/org/apache/commons/math3/linear/RealVector.java
>  Mon May 28 15:39:57 2012
> @@ -202,23 +202,17 @@ public abstract class RealVector {
>     protected void checkIndices(final int start, final int end) {
>         final int dim = getDimension();
>         if ((start < 0) || (start >= dim)) {
> -            throw new OutOfRangeException(LocalizedFormats.INDEX,
> -                                          Integer.valueOf(start),
> -                                          Integer.valueOf(0),
> -                                          Integer.valueOf(dim - 1));
> +            throw new OutOfRangeException(LocalizedFormats.INDEX, start, 0,
> +                                          dim - 1);
>         }
>         if ((end < 0) || (end >= dim)) {
> -            throw new OutOfRangeException(LocalizedFormats.INDEX,
> -                                          Integer.valueOf(end),
> -                                          Integer.valueOf(0),
> -                                          Integer.valueOf(dim - 1));
> +            throw new OutOfRangeException(LocalizedFormats.INDEX, end, 0,
> +                                          dim - 1);
>         }
> -        if (end < start){
> +        if (end < start) {
>             // TODO Use more specific error message
>             throw new 
> NumberIsTooSmallException(LocalizedFormats.INITIAL_ROW_AFTER_FINAL_ROW,
> -                                                Integer.valueOf(end),
> -                                                Integer.valueOf(start),
> -                                                false);
> +                                                end, start, false);
>         }
>     }
>
>
>

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

Reply via email to