Applying Sebb's patch from CODEC-71, improving the memory consumption of isArrayByteBase64(). I've also deprecated the discardWhitespace method as nothing uses it now

git-svn-id: https://svn.apache.org/repos/asf/commons/proper/codec/trunk@672993 13f79535-47bb-0310-9956-ffa450edef68

Author: hen

src/java/org/apache/commons/codec/binary/Base64.java

@@ -476,15 +476,8 @@ public static boolean isBase64(byte octet) {
      *         empty; false, otherwise
      */
     public static boolean isArrayByteBase64(byte[] arrayOctet) {
-
-        arrayOctet = discardWhitespace(arrayOctet);
-
-        int length = arrayOctet.length;
-        if (length == 0) {
-             return true;
-        }
-        for (int i = 0; i < length; i++) {
-            if (!isBase64(arrayOctet[i])) {
+        for (int i = 0; i < arrayOctet.length; i++) {
+            if (!isBase64(arrayOctet[i]) && !isWhiteSpace(arrayOctet[i])) {
                 return false;
             }
         }
@@ -630,6 +623,7 @@ public static byte[] decodeBase64(byte[] base64Data) {
      * @param data
      *            The base-64 encoded data to discard the whitespace from.
      * @return The data, less whitespace (see RFC 2045).
+     * @deprecated This method is no longer needed
      */
     static byte[] discardWhitespace(byte[] data) {
         byte groomedData[] = new byte[data.length];
@@ -654,6 +648,25 @@ static byte[] discardWhitespace(byte[] data) {
         return packedData;
     }
 
+
+    /**
+     * Check if a byte value is whitespace or not.
+     * 
+     * @param byteToCheck the byte to check
+     * @return true if byte is whitespace, false otherwise
+     */
+    private static boolean isWhiteSpace(byte byteToCheck){
+        switch (byteToCheck) {
+        case ' ' :
+        case '\n' :
+        case '\r' :
+        case '\t' :
+            return true;
+        default :
+            return false;
+        }
+    }
+
     /**
      * Discards any characters outside of the base64 alphabet, per the requirements on page 25 of RFC 2045 - "Any
      * characters outside of the base64 alphabet are to be ignored in base64 encoded data."