You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@commons.apache.org by gg...@apache.org on 2011/02/20 22:58:20 UTC

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

Author: ggregory
Date: Sun Feb 20 21:58:19 2011
New Revision: 1072757

URL: http://svn.apache.org/viewvc?rev=1072757&view=rev
Log:
Fix typo in // comment.

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

Modified: commons/proper/codec/trunk/src/java/org/apache/commons/codec/binary/Base32.java
URL: http://svn.apache.org/viewvc/commons/proper/codec/trunk/src/java/org/apache/commons/codec/binary/Base32.java?rev=1072757&r1=1072756&r2=1072757&view=diff
==============================================================================
--- commons/proper/codec/trunk/src/java/org/apache/commons/codec/binary/Base32.java (original)
+++ commons/proper/codec/trunk/src/java/org/apache/commons/codec/binary/Base32.java Sun Feb 20 21:58:19 2011
@@ -243,7 +243,7 @@ public class Base32 extends BaseNCodec {
             if (lineSeparator == null) {
                 throw new IllegalArgumentException("lineLength "+lineLength+" > 0, but lineSeparator is null");
             }
-            // Must be done after initialising the tables
+            // Must be done after initializing the tables
             if (containsAlphabetOrPad(lineSeparator)) {
                 String sep = StringUtils.newStringUtf8(lineSeparator);
                 throw new IllegalArgumentException("lineSeparator must not contain Base32 characters: [" + sep + "]");



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

Posted by sebb <se...@gmail.com>.
On 20 February 2011 21:58,  <gg...@apache.org> wrote:
> Author: ggregory
> Date: Sun Feb 20 21:58:19 2011
> New Revision: 1072757
>
> URL: http://svn.apache.org/viewvc?rev=1072757&view=rev
> Log:
> Fix typo in // comment.
>
> Modified:
>    commons/proper/codec/trunk/src/java/org/apache/commons/codec/binary/Base32.java
>
> Modified: commons/proper/codec/trunk/src/java/org/apache/commons/codec/binary/Base32.java
> URL: http://svn.apache.org/viewvc/commons/proper/codec/trunk/src/java/org/apache/commons/codec/binary/Base32.java?rev=1072757&r1=1072756&r2=1072757&view=diff
> ==============================================================================
> --- commons/proper/codec/trunk/src/java/org/apache/commons/codec/binary/Base32.java (original)
> +++ commons/proper/codec/trunk/src/java/org/apache/commons/codec/binary/Base32.java Sun Feb 20 21:58:19 2011
> @@ -243,7 +243,7 @@ public class Base32 extends BaseNCodec {
>             if (lineSeparator == null) {
>                 throw new IllegalArgumentException("lineLength "+lineLength+" > 0, but lineSeparator is null");
>             }
> -            // Must be done after initialising the tables
> +            // Must be done after initializing the tables

Strictly speaking, that was not a typo - the original spelling was
correct in UK English, the new spelling is US English.

>             if (containsAlphabetOrPad(lineSeparator)) {
>                 String sep = StringUtils.newStringUtf8(lineSeparator);
>                 throw new IllegalArgumentException("lineSeparator must not contain Base32 characters: [" + sep + "]");
>
>
>

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