Hi, Sebb,

Are you sure those are unnecessary?  I cannot compile under Java 1.2.2
without them.  But maybe Java 1.2.2 can still run jar files created by
later compilers when the code has this.

(Java 1.3.1 compiles just fine).


yours,

Julius



On Thu, Jun 19, 2008 at 1:03 PM,  <[EMAIL PROTECTED]> wrote:
> Author: sebb
> Date: Thu Jun 19 13:03:20 2008
> New Revision: 669667
>
> URL: http://svn.apache.org/viewvc?rev=669667&view=rev
> Log:
> Remove some more unnecessary casts
>
> Modified:
>    
> commons/proper/codec/trunk/src/java/org/apache/commons/codec/binary/Base64.java
>
> Modified: 
> commons/proper/codec/trunk/src/java/org/apache/commons/codec/binary/Base64.java
> URL: 
> http://svn.apache.org/viewvc/commons/proper/codec/trunk/src/java/org/apache/commons/codec/binary/Base64.java?rev=669667&r1=669666&r2=669667&view=diff
> ==============================================================================
> --- 
> commons/proper/codec/trunk/src/java/org/apache/commons/codec/binary/Base64.java
>  (original)
> +++ 
> commons/proper/codec/trunk/src/java/org/apache/commons/codec/binary/Base64.java
>  Thu Jun 19 13:03:20 2008
> @@ -61,7 +61,7 @@
>     /**
>      * Byte used to pad output.
>      */
> -    private static final byte PAD = (byte) '=';
> +    private static final byte PAD = '=';
>
>
>     // The static final fields above are used for the original static byte[] 
> methods on Base64.
> @@ -434,7 +434,7 @@
>                 if (b >= 0 && b < base64ToInt.length) {
>                     int result = base64ToInt[b];
>                     if (result >= 0) {
> -                        modulus = (byte) ((++modulus) % 4);
> +                        modulus = (++modulus) % 4;
>                         x = (x << 6) + result;
>                         if (modulus == 0) {
>                             buf[pos++] = (byte) ((x >> 16) & 0xff);
> @@ -614,10 +614,10 @@
>
>         for (int i = 0; i < data.length; i++) {
>             switch (data[i]) {
> -                case (byte) ' ' :
> -                case (byte) '\n' :
> -                case (byte) '\r' :
> -                case (byte) '\t' :
> +                case ' ' :
> +                case '\n' :
> +                case '\r' :
> +                case '\t' :
>                     break;
>                 default :
>                     groomedData[bytesCopied++] = data[i];
>
>
>



-- 
yours,

Julius Davies
250-592-2284 (Home)
250-893-4579 (Mobile)
http://juliusdavies.ca/

---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to