You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@commons.apache.org by se...@apache.org on 2012/09/09 15:05:15 UTC

svn commit: r1382488 - /commons/proper/codec/trunk/src/main/java/org/apache/commons/codec/binary/Base32.java

Author: sebb
Date: Sun Sep  9 13:05:15 2012
New Revision: 1382488

URL: http://svn.apache.org/viewvc?rev=1382488&view=rev
Log:
Case 0 is exercised by tests, so is correct

Modified:
    commons/proper/codec/trunk/src/main/java/org/apache/commons/codec/binary/Base32.java

Modified: commons/proper/codec/trunk/src/main/java/org/apache/commons/codec/binary/Base32.java
URL: http://svn.apache.org/viewvc/commons/proper/codec/trunk/src/main/java/org/apache/commons/codec/binary/Base32.java?rev=1382488&r1=1382487&r2=1382488&view=diff
==============================================================================
--- commons/proper/codec/trunk/src/main/java/org/apache/commons/codec/binary/Base32.java (original)
+++ commons/proper/codec/trunk/src/main/java/org/apache/commons/codec/binary/Base32.java Sun Sep  9 13:05:15 2012
@@ -389,7 +389,7 @@ public class Base32 extends BaseNCodec {
             final byte[] buffer = ensureBufferSize(encodeSize, context);
             final int savedPos = context.pos;
             switch (context.modulus) { // % 5
-                case 0 : // TODO - correct?
+                case 0 :
                     break;
                 case 1 : // Only 1 octet; take top 5 bits then remainder
                     buffer[context.pos++] = encodeTable[(int)(context.lbitWorkArea >> 3) & MASK_5BITS]; // 8-1*5 = 3