You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@commons.apache.org by pa...@apache.org on 2018/11/20 17:21:47 UTC

[4/6] [lang] Make whitespace use after tokens consistent and add a checkstyle rule to enforce it.

Make whitespace use after tokens consistent and add a checkstyle rule to enforce it.


Project: http://git-wip-us.apache.org/repos/asf/commons-lang/repo
Commit: http://git-wip-us.apache.org/repos/asf/commons-lang/commit/89cbfa54
Tree: http://git-wip-us.apache.org/repos/asf/commons-lang/tree/89cbfa54
Diff: http://git-wip-us.apache.org/repos/asf/commons-lang/diff/89cbfa54

Branch: refs/heads/master
Commit: 89cbfa54b041541077b8158b667ee3830a8abe5c
Parents: c4d0dbc
Author: pascalschumacher <pa...@gmx.net>
Authored: Sun Nov 18 11:59:34 2018 +0100
Committer: pascalschumacher <pa...@gmx.net>
Committed: Mon Nov 19 22:04:35 2018 +0100

----------------------------------------------------------------------
 checkstyle.xml                                  |   1 +
 .../org/apache/commons/lang3/ArrayUtils.java    |  34 +-
 .../java/org/apache/commons/lang3/CharSet.java  |   2 +-
 .../org/apache/commons/lang3/Conversion.java    |   8 +-
 .../org/apache/commons/lang3/LocaleUtils.java   |   6 +-
 .../commons/lang3/SerializationUtils.java       |   2 +-
 .../apache/commons/lang3/StringEscapeUtils.java |   4 +-
 .../org/apache/commons/lang3/StringUtils.java   | 106 ++--
 .../org/apache/commons/lang3/ThreadUtils.java   |  16 +-
 .../commons/lang3/builder/ToStringStyle.java    |  20 +-
 .../lang3/concurrent/TimedSemaphore.java        |   2 +-
 .../lang3/exception/ContextedException.java     |   2 +-
 .../exception/ContextedRuntimeException.java    |   2 +-
 .../exception/DefaultExceptionContext.java      |   2 +-
 .../commons/lang3/math/IEEE754rUtils.java       |  16 +-
 .../apache/commons/lang3/math/NumberUtils.java  |  30 +-
 .../org/apache/commons/lang3/package-info.java  |   2 +-
 .../commons/lang3/reflect/ConstructorUtils.java |   2 +-
 .../commons/lang3/reflect/MethodUtils.java      |  16 +-
 .../apache/commons/lang3/reflect/TypeUtils.java |   2 +-
 .../lang3/text/ExtendedMessageFormat.java       |   4 +-
 .../apache/commons/lang3/text/StrBuilder.java   |   8 +-
 .../commons/lang3/text/StrSubstitutor.java      |   4 +-
 .../apache/commons/lang3/text/WordUtils.java    |   6 +-
 .../text/translate/AggregateTranslator.java     |   2 +-
 .../lang3/text/translate/EntityArrays.java      | 126 ++---
 .../text/translate/NumericEntityEscaper.java    |   2 +-
 .../text/translate/NumericEntityUnescaper.java  |  20 +-
 .../lang3/text/translate/OctalUnescaper.java    |   6 +-
 .../lang3/text/translate/UnicodeEscaper.java    |   2 +-
 .../apache/commons/lang3/time/DateUtils.java    |   4 +-
 .../commons/lang3/time/DurationFormatUtils.java |   4 +-
 .../commons/lang3/time/FastDateParser.java      |  16 +-
 .../commons/lang3/time/FastDatePrinter.java     |  20 +-
 .../apache/commons/lang3/time/FormatCache.java  |  12 +-
 .../apache/commons/lang3/ArrayUtilsAddTest.java |  52 +-
 .../commons/lang3/ArrayUtilsInsertTest.java     | 144 ++---
 .../commons/lang3/ArrayUtilsRemoveTest.java     |   4 +-
 .../apache/commons/lang3/ArrayUtilsTest.java    |  14 +-
 .../apache/commons/lang3/BooleanUtilsTest.java  |   2 +-
 .../commons/lang3/CharSequenceUtilsTest.java    |  10 +-
 .../org/apache/commons/lang3/CharSetTest.java   |   4 +-
 .../apache/commons/lang3/ClassUtilsTest.java    |  12 +-
 .../apache/commons/lang3/ConversionTest.java    | 534 +++++++++----------
 .../commons/lang3/HashSetvBitSetTest.java       |   4 +-
 .../apache/commons/lang3/ObjectUtilsTest.java   |  12 +-
 .../commons/lang3/RandomStringUtilsTest.java    |  30 +-
 .../commons/lang3/SerializationUtilsTest.java   |   2 +-
 .../commons/lang3/StringEscapeUtilsTest.java    |   6 +-
 .../lang3/StringUtilsEqualsIndexOfTest.java     |  22 +-
 .../lang3/StringUtilsStartsEndsWithTest.java    |  16 +-
 .../commons/lang3/StringUtilsSubstringTest.java |   2 +-
 .../apache/commons/lang3/StringUtilsTest.java   |   2 +-
 .../apache/commons/lang3/ThreadUtilsTest.java   |  32 +-
 .../lang3/builder/CompareToBuilderTest.java     |  20 +-
 .../lang3/builder/JsonToStringStyleTest.java    |  16 +-
 ...ionToStringBuilderExcludeNullValuesTest.java |  10 +-
 .../lang3/builder/ToStringBuilderTest.java      |   6 +-
 .../ContextedRuntimeExceptionTest.java          |   4 +-
 .../apache/commons/lang3/math/FractionTest.java |  18 +-
 .../commons/lang3/math/NumberUtilsTest.java     | 130 ++---
 .../commons/lang3/mutable/MutableByteTest.java  |  12 +-
 .../lang3/mutable/MutableDoubleTest.java        |   2 +-
 .../commons/lang3/mutable/MutableFloatTest.java |   2 +-
 .../commons/lang3/mutable/MutableIntTest.java   |   2 +-
 .../commons/lang3/mutable/MutableLongTest.java  |   2 +-
 .../commons/lang3/mutable/MutableShortTest.java |   6 +-
 .../lang3/reflect/ConstructorUtilsTest.java     |   4 +-
 .../commons/lang3/reflect/FieldUtilsTest.java   |   2 +-
 .../commons/lang3/reflect/TypeUtilsTest.java    |   2 +-
 .../lang3/text/ExtendedMessageFormatTest.java   |   2 +-
 .../lang3/text/StrBuilderAppendInsertTest.java  |   2 +-
 .../commons/lang3/text/StrBuilderTest.java      |  48 +-
 .../commons/lang3/text/StrSubstitutorTest.java  |   2 +-
 .../commons/lang3/text/StrTokenizerTest.java    |  16 +-
 .../commons/lang3/text/WordUtilsTest.java       |   2 +-
 .../lang3/text/translate/EntityArraysTest.java  |   8 +-
 .../lang3/time/DateUtilsFragmentTest.java       |   8 +-
 .../commons/lang3/time/DateUtilsTest.java       |   2 +-
 .../lang3/time/DurationFormatUtilsTest.java     |   6 +-
 .../commons/lang3/time/FastDateFormatTest.java  |   8 +-
 .../commons/lang3/time/FastDateParserTest.java  |  42 +-
 82 files changed, 901 insertions(+), 898 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/commons-lang/blob/89cbfa54/checkstyle.xml
----------------------------------------------------------------------
diff --git a/checkstyle.xml b/checkstyle.xml
index 9ba1bcc..057c1aa 100644
--- a/checkstyle.xml
+++ b/checkstyle.xml
@@ -55,5 +55,6 @@ limitations under the License.
     <module name="LeftCurly"/>
     <module name="NeedBraces"/>
     <module name="RightCurly"/>
+    <module name="WhitespaceAfter"/>
  </module>
 </module>

http://git-wip-us.apache.org/repos/asf/commons-lang/blob/89cbfa54/src/main/java/org/apache/commons/lang3/ArrayUtils.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/commons/lang3/ArrayUtils.java b/src/main/java/org/apache/commons/lang3/ArrayUtils.java
index 4ff2553..eb44db4 100644
--- a/src/main/java/org/apache/commons/lang3/ArrayUtils.java
+++ b/src/main/java/org/apache/commons/lang3/ArrayUtils.java
@@ -244,7 +244,7 @@ public class ArrayUtils {
         for (int i = 0; i < array.length; i++) {
             final Object object = array[i];
             if (object instanceof Map.Entry<?, ?>) {
-                final Map.Entry<?,?> entry = (Map.Entry<?,?>) object;
+                final Map.Entry<?, ?> entry = (Map.Entry<?, ?>) object;
                 map.put(entry.getKey(), entry.getValue());
             } else if (object instanceof Object[]) {
                 final Object[] entry = (Object[]) object;
@@ -275,7 +275,7 @@ public class ArrayUtils {
         return new T[size]; // compiler error here
     }
     public static &lt;T&gt; T[] createAnArray(int size) {
-        return (T[])new Object[size]; // ClassCastException at runtime
+        return (T[]) new Object[size]; // ClassCastException at runtime
     }
      * </pre>
      *
@@ -877,7 +877,7 @@ public class ArrayUtils {
      * {@code Date}, the following usage is envisaged:
      *
      * <pre>
-     * Date[] someDates = (Date[])ArrayUtils.subarray(allDates, 2, 5);
+     * Date[] someDates = (Date[]) ArrayUtils.subarray(allDates, 2, 5);
      * </pre>
      *
      * @param <T> the component type of the array
@@ -4748,19 +4748,19 @@ public class ArrayUtils {
         }
         final Class<?> ct = array.getClass().getComponentType();
         final Class<?> pt = ClassUtils.wrapperToPrimitive(ct);
-        if(Integer.TYPE.equals(pt)) {
+        if (Integer.TYPE.equals(pt)) {
             return toPrimitive((Integer[]) array);
         }
-        if(Long.TYPE.equals(pt)) {
+        if (Long.TYPE.equals(pt)) {
             return toPrimitive((Long[]) array);
         }
-        if(Short.TYPE.equals(pt)) {
+        if (Short.TYPE.equals(pt)) {
             return toPrimitive((Short[]) array);
         }
-        if(Double.TYPE.equals(pt)) {
+        if (Double.TYPE.equals(pt)) {
             return toPrimitive((Double[]) array);
         }
-        if(Float.TYPE.equals(pt)) {
+        if (Float.TYPE.equals(pt)) {
             return toPrimitive((Float[]) array);
         }
         return array;
@@ -5387,7 +5387,7 @@ public class ArrayUtils {
      * @since 2.1
      */
     public static boolean[] add(final boolean[] array, final boolean element) {
-        final boolean[] newArray = (boolean[])copyArrayGrow1(array, Boolean.TYPE);
+        final boolean[] newArray = (boolean[]) copyArrayGrow1(array, Boolean.TYPE);
         newArray[newArray.length - 1] = element;
         return newArray;
     }
@@ -5414,7 +5414,7 @@ public class ArrayUtils {
      * @since 2.1
      */
     public static byte[] add(final byte[] array, final byte element) {
-        final byte[] newArray = (byte[])copyArrayGrow1(array, Byte.TYPE);
+        final byte[] newArray = (byte[]) copyArrayGrow1(array, Byte.TYPE);
         newArray[newArray.length - 1] = element;
         return newArray;
     }
@@ -5441,7 +5441,7 @@ public class ArrayUtils {
      * @since 2.1
      */
     public static char[] add(final char[] array, final char element) {
-        final char[] newArray = (char[])copyArrayGrow1(array, Character.TYPE);
+        final char[] newArray = (char[]) copyArrayGrow1(array, Character.TYPE);
         newArray[newArray.length - 1] = element;
         return newArray;
     }
@@ -5468,7 +5468,7 @@ public class ArrayUtils {
      * @since 2.1
      */
     public static double[] add(final double[] array, final double element) {
-        final double[] newArray = (double[])copyArrayGrow1(array, Double.TYPE);
+        final double[] newArray = (double[]) copyArrayGrow1(array, Double.TYPE);
         newArray[newArray.length - 1] = element;
         return newArray;
     }
@@ -5495,7 +5495,7 @@ public class ArrayUtils {
      * @since 2.1
      */
     public static float[] add(final float[] array, final float element) {
-        final float[] newArray = (float[])copyArrayGrow1(array, Float.TYPE);
+        final float[] newArray = (float[]) copyArrayGrow1(array, Float.TYPE);
         newArray[newArray.length - 1] = element;
         return newArray;
     }
@@ -5522,7 +5522,7 @@ public class ArrayUtils {
      * @since 2.1
      */
     public static int[] add(final int[] array, final int element) {
-        final int[] newArray = (int[])copyArrayGrow1(array, Integer.TYPE);
+        final int[] newArray = (int[]) copyArrayGrow1(array, Integer.TYPE);
         newArray[newArray.length - 1] = element;
         return newArray;
     }
@@ -5549,7 +5549,7 @@ public class ArrayUtils {
      * @since 2.1
      */
     public static long[] add(final long[] array, final long element) {
-        final long[] newArray = (long[])copyArrayGrow1(array, Long.TYPE);
+        final long[] newArray = (long[]) copyArrayGrow1(array, Long.TYPE);
         newArray[newArray.length - 1] = element;
         return newArray;
     }
@@ -8681,8 +8681,8 @@ public class ArrayUtils {
      * @return Whether the given index is safely-accessible in the given array
      * @since 3.8
      */
-    public static <T> boolean isArrayIndexValid(T[] array, int index){
-        if(getLength(array) == 0 || array.length <= index){
+    public static <T> boolean isArrayIndexValid(T[] array, int index) {
+        if (getLength(array) == 0 || array.length <= index) {
             return false;
         }
 

http://git-wip-us.apache.org/repos/asf/commons-lang/blob/89cbfa54/src/main/java/org/apache/commons/lang3/CharSet.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/commons/lang3/CharSet.java b/src/main/java/org/apache/commons/lang3/CharSet.java
index 6cf0246..21e7e17 100644
--- a/src/main/java/org/apache/commons/lang3/CharSet.java
+++ b/src/main/java/org/apache/commons/lang3/CharSet.java
@@ -132,7 +132,7 @@ public class CharSet implements Serializable {
      * <p>There are two ways to add a literal negation character ({@code ^}):</p>
      * <ul>
      *     <li>As the last character in a string, e.g. {@code CharSet.getInstance("a-z^")}</li>
-     *     <li>As a separate element, e.g. {@code CharSet.getInstance("^","a-z")}</li>
+     *     <li>As a separate element, e.g. {@code CharSet.getInstance("^", "a-z")}</li>
      * </ul>
      *
      * <p>Examples using the negation character:</p>

http://git-wip-us.apache.org/repos/asf/commons-lang/blob/89cbfa54/src/main/java/org/apache/commons/lang3/Conversion.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/commons/lang3/Conversion.java b/src/main/java/org/apache/commons/lang3/Conversion.java
index 4e11776..de48081 100644
--- a/src/main/java/org/apache/commons/lang3/Conversion.java
+++ b/src/main/java/org/apache/commons/lang3/Conversion.java
@@ -359,7 +359,7 @@ public class Conversion {
      * ordering.
      * </p>
      * <p>
-     * (1, 0, 0, 0) is converted as follow: '8' (1,0,0,1,1,0,1,0) with srcPos = 3 is converted
+     * (1, 0, 0, 0) is converted as follow: '8' (1, 0, 0, 1, 1, 0, 1, 0) with srcPos = 3 is converted
      * to 'D'
      * </p>
      *
@@ -407,7 +407,7 @@ public class Conversion {
      * bit ordering to a hexadecimal digit.
      * </p>
      * <p>
-     * (1, 0, 0, 0) is converted as follow: '8' (1,0,0,0,0,0,0,0, 0,0,0,0,0,1,0,0) is converted
+     * (1, 0, 0, 0) is converted as follow: '8' (1, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 1, 0, 0) is converted
      * to '4'
      * </p>
      *
@@ -426,8 +426,8 @@ public class Conversion {
      * hexadecimal digit.
      * </p>
      * <p>
-     * (1, 0, 0, 0) with srcPos = 0 is converted as follow: '8' (1,0,0,0,0,0,0,0,
-     * 0,0,0,1,0,1,0,0) with srcPos = 2 is converted to '5'
+     * (1, 0, 0, 0) with srcPos = 0 is converted as follow: '8' (1, 0, 0, 0, 0, 0, 0, 0,
+     * 0, 0, 0, 1, 0, 1, 0, 0) with srcPos = 2 is converted to '5'
      * </p>
      *
      * @param src the binary to convert

http://git-wip-us.apache.org/repos/asf/commons-lang/blob/89cbfa54/src/main/java/org/apache/commons/lang3/LocaleUtils.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/commons/lang3/LocaleUtils.java b/src/main/java/org/apache/commons/lang3/LocaleUtils.java
index d5b5043..703b336 100644
--- a/src/main/java/org/apache/commons/lang3/LocaleUtils.java
+++ b/src/main/java/org/apache/commons/lang3/LocaleUtils.java
@@ -194,8 +194,8 @@ public class LocaleUtils {
      * a locale search.</p>
      *
      * <pre>
-     * localeLookupList(Locale("fr","CA","xxx"))
-     *   = [Locale("fr","CA","xxx"), Locale("fr","CA"), Locale("fr")]
+     * localeLookupList(Locale("fr", "CA", "xxx"))
+     *   = [Locale("fr", "CA", "xxx"), Locale("fr", "CA"), Locale("fr")]
      * </pre>
      *
      * @param locale  the locale to start from
@@ -212,7 +212,7 @@ public class LocaleUtils {
      *
      * <pre>
      * localeLookupList(Locale("fr", "CA", "xxx"), Locale("en"))
-     *   = [Locale("fr","CA","xxx"), Locale("fr","CA"), Locale("fr"), Locale("en"]
+     *   = [Locale("fr", "CA", "xxx"), Locale("fr", "CA"), Locale("fr"), Locale("en"]
      * </pre>
      *
      * <p>The result list begins with the most specific locale, then the

http://git-wip-us.apache.org/repos/asf/commons-lang/blob/89cbfa54/src/main/java/org/apache/commons/lang3/SerializationUtils.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/commons/lang3/SerializationUtils.java b/src/main/java/org/apache/commons/lang3/SerializationUtils.java
index abe0331..8446f58 100644
--- a/src/main/java/org/apache/commons/lang3/SerializationUtils.java
+++ b/src/main/java/org/apache/commons/lang3/SerializationUtils.java
@@ -134,7 +134,7 @@ public class SerializationUtils {
      */
     public static void serialize(final Serializable obj, final OutputStream outputStream) {
         Validate.isTrue(outputStream != null, "The OutputStream must not be null");
-        try (ObjectOutputStream out = new ObjectOutputStream(outputStream)){
+        try (ObjectOutputStream out = new ObjectOutputStream(outputStream)) {
             out.writeObject(obj);
         } catch (final IOException ex) {
             throw new SerializationException(ex);

http://git-wip-us.apache.org/repos/asf/commons-lang/blob/89cbfa54/src/main/java/org/apache/commons/lang3/StringEscapeUtils.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/commons/lang3/StringEscapeUtils.java b/src/main/java/org/apache/commons/lang3/StringEscapeUtils.java
index ce7eec5..64f1d5d 100644
--- a/src/main/java/org/apache/commons/lang3/StringEscapeUtils.java
+++ b/src/main/java/org/apache/commons/lang3/StringEscapeUtils.java
@@ -260,7 +260,7 @@ public class StringEscapeUtils {
         @Override
         public int translate(final CharSequence input, final int index, final Writer out) throws IOException {
 
-            if(index != 0) {
+            if (index != 0) {
                 throw new IllegalStateException("CsvEscaper should never reach the [1] index");
             }
 
@@ -393,7 +393,7 @@ public class StringEscapeUtils {
         @Override
         public int translate(final CharSequence input, final int index, final Writer out) throws IOException {
 
-            if(index != 0) {
+            if (index != 0) {
                 throw new IllegalStateException("CsvUnescaper should never reach the [1] index");
             }
 

http://git-wip-us.apache.org/repos/asf/commons-lang/blob/89cbfa54/src/main/java/org/apache/commons/lang3/StringUtils.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/commons/lang3/StringUtils.java b/src/main/java/org/apache/commons/lang3/StringUtils.java
index 12dfcdc..a21e5a9 100644
--- a/src/main/java/org/apache/commons/lang3/StringUtils.java
+++ b/src/main/java/org/apache/commons/lang3/StringUtils.java
@@ -256,7 +256,7 @@ public class StringUtils {
       if (ArrayUtils.isEmpty(css)) {
         return false;
       }
-      for (final CharSequence cs : css){
+      for (final CharSequence cs : css) {
         if (isEmpty(cs)) {
           return true;
         }
@@ -400,7 +400,7 @@ public class StringUtils {
       if (ArrayUtils.isEmpty(css)) {
         return false;
       }
-      for (final CharSequence cs : css){
+      for (final CharSequence cs : css) {
         if (isBlank(cs)) {
           return true;
         }
@@ -952,10 +952,10 @@ public class StringUtils {
      */
     // See also Lucene's ASCIIFoldingFilter (Lucene 2.9) that replaces accented characters by their unaccented equivalent (and uncommitted bug fix: https://issues.apache.org/jira/browse/LUCENE-1343?focusedCommentId=12858907&page=com.atlassian.jira.plugin.system.issuetabpanels%3Acomment-tabpanel#action_12858907).
     public static String stripAccents(final String input) {
-        if(input == null) {
+        if (input == null) {
             return null;
         }
-        final Pattern pattern = Pattern.compile("\\p{InCombiningDiacriticalMarks}+");//$NON-NLS-1$
+        final Pattern pattern = Pattern.compile("\\p{InCombiningDiacriticalMarks}+"); //$NON-NLS-1$
         final StringBuilder decomposed = new StringBuilder(Normalizer.normalize(input, Normalizer.Form.NFD));
         convertRemainingAccentCharacters(decomposed);
         // Note that this doesn't correctly remove ligatures...
@@ -1488,9 +1488,9 @@ public class StringUtils {
      *
      * <p>Matches may overlap:</p>
      * <pre>
-     * StringUtils.ordinalIndexOf("ababab","aba", 1)   = 0
-     * StringUtils.ordinalIndexOf("ababab","aba", 2)   = 2
-     * StringUtils.ordinalIndexOf("ababab","aba", 3)   = -1
+     * StringUtils.ordinalIndexOf("ababab", "aba", 1)   = 0
+     * StringUtils.ordinalIndexOf("ababab", "aba", 2)   = 2
+     * StringUtils.ordinalIndexOf("ababab", "aba", 3)   = -1
      *
      * StringUtils.ordinalIndexOf("abababab", "abab", 1) = 0
      * StringUtils.ordinalIndexOf("abababab", "abab", 2) = 2
@@ -1530,7 +1530,7 @@ public class StringUtils {
      * @return the n-th index of the search CharSequence,
      *  {@code -1} ({@code INDEX_NOT_FOUND}) if no match or {@code null} string input
      */
-    // Shared code between ordinalIndexOf(String,String,int) and lastOrdinalIndexOf(String,String,int)
+    // Shared code between ordinalIndexOf(String, String, int) and lastOrdinalIndexOf(String, String, int)
     private static int ordinalIndexOf(final CharSequence str, final CharSequence searchStr, final int ordinal, final boolean lastIndex) {
         if (str == null || searchStr == null || ordinal <= 0) {
             return INDEX_NOT_FOUND;
@@ -2057,13 +2057,13 @@ public class StringUtils {
      * A {@code null} or zero length search array will return {@code -1}.</p>
      *
      * <pre>
-     * StringUtils.indexOfAny(null, *)                = -1
-     * StringUtils.indexOfAny("", *)                  = -1
-     * StringUtils.indexOfAny(*, null)                = -1
-     * StringUtils.indexOfAny(*, [])                  = -1
-     * StringUtils.indexOfAny("zzabyycdxx",['z','a']) = 0
-     * StringUtils.indexOfAny("zzabyycdxx",['b','y']) = 3
-     * StringUtils.indexOfAny("aba", ['z'])           = -1
+     * StringUtils.indexOfAny(null, *)                  = -1
+     * StringUtils.indexOfAny("", *)                    = -1
+     * StringUtils.indexOfAny(*, null)                  = -1
+     * StringUtils.indexOfAny(*, [])                    = -1
+     * StringUtils.indexOfAny("zzabyycdxx", ['z', 'a']) = 0
+     * StringUtils.indexOfAny("zzabyycdxx", ['b', 'y']) = 3
+     * StringUtils.indexOfAny("aba", ['z'])             = -1
      * </pre>
      *
      * @param cs  the CharSequence to check, may be null
@@ -2112,7 +2112,7 @@ public class StringUtils {
      * StringUtils.indexOfAny(*, "")              = -1
      * StringUtils.indexOfAny("zzabyycdxx", "za") = 0
      * StringUtils.indexOfAny("zzabyycdxx", "by") = 3
-     * StringUtils.indexOfAny("aba","z")          = -1
+     * StringUtils.indexOfAny("aba", "z")         = -1
      * </pre>
      *
      * @param cs  the CharSequence to check, may be null
@@ -2138,14 +2138,14 @@ public class StringUtils {
      * A {@code null} or zero length search array will return {@code false}.</p>
      *
      * <pre>
-     * StringUtils.containsAny(null, *)                = false
-     * StringUtils.containsAny("", *)                  = false
-     * StringUtils.containsAny(*, null)                = false
-     * StringUtils.containsAny(*, [])                  = false
-     * StringUtils.containsAny("zzabyycdxx",['z','a']) = true
-     * StringUtils.containsAny("zzabyycdxx",['b','y']) = true
-     * StringUtils.containsAny("zzabyycdxx",['z','y']) = true
-     * StringUtils.containsAny("aba", ['z'])           = false
+     * StringUtils.containsAny(null, *)                  = false
+     * StringUtils.containsAny("", *)                    = false
+     * StringUtils.containsAny(*, null)                  = false
+     * StringUtils.containsAny(*, [])                    = false
+     * StringUtils.containsAny("zzabyycdxx", ['z', 'a']) = true
+     * StringUtils.containsAny("zzabyycdxx", ['b', 'y']) = true
+     * StringUtils.containsAny("zzabyycdxx", ['z', 'y']) = true
+     * StringUtils.containsAny("aba", ['z'])             = false
      * </pre>
      *
      * @param cs  the CharSequence to check, may be null
@@ -2205,7 +2205,7 @@ public class StringUtils {
      * StringUtils.containsAny("zzabyycdxx", "zy")    = true
      * StringUtils.containsAny("zzabyycdxx", "\tx")   = true
      * StringUtils.containsAny("zzabyycdxx", "$.#yF") = true
-     * StringUtils.containsAny("aba","z")             = false
+     * StringUtils.containsAny("aba", "z")            = false
      * </pre>
      *
      * @param cs
@@ -2327,7 +2327,7 @@ public class StringUtils {
      * StringUtils.indexOfAnyBut(*, "")              = -1
      * StringUtils.indexOfAnyBut("zzabyycdxx", "za") = 3
      * StringUtils.indexOfAnyBut("zzabyycdxx", "")   = -1
-     * StringUtils.indexOfAnyBut("aba","ab")         = -1
+     * StringUtils.indexOfAnyBut("aba", "ab")        = -1
      * </pre>
      *
      * @param seq  the CharSequence to check, may be null
@@ -2523,16 +2523,16 @@ public class StringUtils {
      * null. This method uses {@link String#indexOf(String)} if possible.</p>
      *
      * <pre>
-     * StringUtils.indexOfAny(null, *)                     = -1
-     * StringUtils.indexOfAny(*, null)                     = -1
-     * StringUtils.indexOfAny(*, [])                       = -1
-     * StringUtils.indexOfAny("zzabyycdxx", ["ab","cd"])   = 2
-     * StringUtils.indexOfAny("zzabyycdxx", ["cd","ab"])   = 2
-     * StringUtils.indexOfAny("zzabyycdxx", ["mn","op"])   = -1
-     * StringUtils.indexOfAny("zzabyycdxx", ["zab","aby"]) = 1
-     * StringUtils.indexOfAny("zzabyycdxx", [""])          = 0
-     * StringUtils.indexOfAny("", [""])                    = 0
-     * StringUtils.indexOfAny("", ["a"])                   = -1
+     * StringUtils.indexOfAny(null, *)                      = -1
+     * StringUtils.indexOfAny(*, null)                      = -1
+     * StringUtils.indexOfAny(*, [])                        = -1
+     * StringUtils.indexOfAny("zzabyycdxx", ["ab", "cd"])   = 2
+     * StringUtils.indexOfAny("zzabyycdxx", ["cd", "ab"])   = 2
+     * StringUtils.indexOfAny("zzabyycdxx", ["mn", "op"])   = -1
+     * StringUtils.indexOfAny("zzabyycdxx", ["zab", "aby"]) = 1
+     * StringUtils.indexOfAny("zzabyycdxx", [""])           = 0
+     * StringUtils.indexOfAny("", [""])                     = 0
+     * StringUtils.indexOfAny("", ["a"])                    = -1
      * </pre>
      *
      * @param str  the CharSequence to check, may be null
@@ -2576,15 +2576,15 @@ public class StringUtils {
      * if {@code str} is not null. This method uses {@link String#indexOf(String)} if possible</p>
      *
      * <pre>
-     * StringUtils.lastIndexOfAny(null, *)                   = -1
-     * StringUtils.lastIndexOfAny(*, null)                   = -1
-     * StringUtils.lastIndexOfAny(*, [])                     = -1
-     * StringUtils.lastIndexOfAny(*, [null])                 = -1
-     * StringUtils.lastIndexOfAny("zzabyycdxx", ["ab","cd"]) = 6
-     * StringUtils.lastIndexOfAny("zzabyycdxx", ["cd","ab"]) = 6
-     * StringUtils.lastIndexOfAny("zzabyycdxx", ["mn","op"]) = -1
-     * StringUtils.lastIndexOfAny("zzabyycdxx", ["mn","op"]) = -1
-     * StringUtils.lastIndexOfAny("zzabyycdxx", ["mn",""])   = 10
+     * StringUtils.lastIndexOfAny(null, *)                    = -1
+     * StringUtils.lastIndexOfAny(*, null)                    = -1
+     * StringUtils.lastIndexOfAny(*, [])                      = -1
+     * StringUtils.lastIndexOfAny(*, [null])                  = -1
+     * StringUtils.lastIndexOfAny("zzabyycdxx", ["ab", "cd"]) = 6
+     * StringUtils.lastIndexOfAny("zzabyycdxx", ["cd", "ab"]) = 6
+     * StringUtils.lastIndexOfAny("zzabyycdxx", ["mn", "op"]) = -1
+     * StringUtils.lastIndexOfAny("zzabyycdxx", ["mn", "op"]) = -1
+     * StringUtils.lastIndexOfAny("zzabyycdxx", ["mn", ""])   = 10
      * </pre>
      *
      * @param str  the CharSequence to check, may be null
@@ -4910,7 +4910,7 @@ public class StringUtils {
         if (isEmpty(str) || isEmpty(remove)) {
             return str;
         }
-        if (str.startsWith(remove)){
+        if (str.startsWith(remove)) {
             return str.substring(remove.length());
         }
         return str;
@@ -6161,7 +6161,7 @@ public class StringUtils {
      */
     @Deprecated
     public static String chomp(final String str, final String separator) {
-        return removeEnd(str,separator);
+        return removeEnd(str, separator);
     }
 
     // Chopping
@@ -6289,7 +6289,7 @@ public class StringUtils {
      * @since 2.5
      */
     public static String repeat(final String str, final String separator, final int repeat) {
-        if(str == null || separator == null) {
+        if (str == null || separator == null) {
             return repeat(str, repeat);
         }
         // given that repeat(String, int) is quite optimized, better to rely on it than try and splice this into it
@@ -8176,8 +8176,8 @@ public class StringUtils {
      * <pre>
      * StringUtils.getLevenshteinDistance(null, *)             = IllegalArgumentException
      * StringUtils.getLevenshteinDistance(*, null)             = IllegalArgumentException
-     * StringUtils.getLevenshteinDistance("","")               = 0
-     * StringUtils.getLevenshteinDistance("","a")              = 1
+     * StringUtils.getLevenshteinDistance("", "")              = 0
+     * StringUtils.getLevenshteinDistance("", "a")             = 1
      * StringUtils.getLevenshteinDistance("aaapppp", "")       = 7
      * StringUtils.getLevenshteinDistance("frog", "fog")       = 1
      * StringUtils.getLevenshteinDistance("fly", "ant")        = 3
@@ -8268,7 +8268,7 @@ public class StringUtils {
      * StringUtils.getLevenshteinDistance(null, *, *)             = IllegalArgumentException
      * StringUtils.getLevenshteinDistance(*, null, *)             = IllegalArgumentException
      * StringUtils.getLevenshteinDistance(*, *, -1)               = IllegalArgumentException
-     * StringUtils.getLevenshteinDistance("","", 0)               = 0
+     * StringUtils.getLevenshteinDistance("", "", 0)              = 0
      * StringUtils.getLevenshteinDistance("aaapppp", "", 8)       = 7
      * StringUtils.getLevenshteinDistance("aaapppp", "", 7)       = 7
      * StringUtils.getLevenshteinDistance("aaapppp", "", 6))      = -1
@@ -8431,8 +8431,8 @@ public class StringUtils {
      *
      * <pre>
      * StringUtils.getJaroWinklerDistance(null, null)          = IllegalArgumentException
-     * StringUtils.getJaroWinklerDistance("","")               = 0.0
-     * StringUtils.getJaroWinklerDistance("","a")              = 0.0
+     * StringUtils.getJaroWinklerDistance("", "")              = 0.0
+     * StringUtils.getJaroWinklerDistance("", "a")             = 0.0
      * StringUtils.getJaroWinklerDistance("aaapppp", "")       = 0.0
      * StringUtils.getJaroWinklerDistance("frog", "fog")       = 0.93
      * StringUtils.getJaroWinklerDistance("fly", "ant")        = 0.0

http://git-wip-us.apache.org/repos/asf/commons-lang/blob/89cbfa54/src/main/java/org/apache/commons/lang3/ThreadUtils.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/commons/lang3/ThreadUtils.java b/src/main/java/org/apache/commons/lang3/ThreadUtils.java
index caf6fa0..16986e0 100644
--- a/src/main/java/org/apache/commons/lang3/ThreadUtils.java
+++ b/src/main/java/org/apache/commons/lang3/ThreadUtils.java
@@ -52,7 +52,7 @@ public class ThreadUtils {
     public static Thread findThreadById(final long threadId, final ThreadGroup threadGroup) {
         Validate.isTrue(threadGroup != null, "The thread group must not be null");
         final Thread thread = findThreadById(threadId);
-        if(thread != null && threadGroup.equals(thread.getThreadGroup())) {
+        if (thread != null && threadGroup.equals(thread.getThreadGroup())) {
             return thread;
         }
         return null;
@@ -75,7 +75,7 @@ public class ThreadUtils {
     public static Thread findThreadById(final long threadId, final String threadGroupName) {
         Validate.isTrue(threadGroupName != null, "The thread group name must not be null");
         final Thread thread = findThreadById(threadId);
-        if(thread != null && thread.getThreadGroup() != null && thread.getThreadGroup().getName().equals(threadGroupName)) {
+        if (thread != null && thread.getThreadGroup() != null && thread.getThreadGroup().getName().equals(threadGroupName)) {
             return thread;
         }
         return null;
@@ -119,13 +119,13 @@ public class ThreadUtils {
 
         final Collection<ThreadGroup> threadGroups = findThreadGroups(new NamePredicate(threadGroupName));
 
-        if(threadGroups.isEmpty()) {
+        if (threadGroups.isEmpty()) {
             return Collections.emptyList();
         }
 
         final Collection<Thread> result = new ArrayList<>();
         final NamePredicate threadNamePredicate = new NamePredicate(threadName);
-        for(final ThreadGroup group : threadGroups) {
+        for (final ThreadGroup group : threadGroups) {
             result.addAll(findThreads(group, false, threadNamePredicate));
         }
         return Collections.unmodifiableCollection(result);
@@ -170,7 +170,7 @@ public class ThreadUtils {
      */
     public static ThreadGroup getSystemThreadGroup() {
         ThreadGroup threadGroup = Thread.currentThread().getThreadGroup();
-        while(threadGroup.getParent() != null) {
+        while (threadGroup.getParent() != null) {
             threadGroup = threadGroup.getParent();
         }
         return threadGroup;
@@ -431,11 +431,11 @@ public class ThreadUtils {
             threadGroups = new ThreadGroup[count + (count / 2) + 1]; //slightly grow the array size
             count = group.enumerate(threadGroups, recurse);
             //return value of enumerate() must be strictly less than the array size according to javadoc
-        } while(count >= threadGroups.length);
+        } while (count >= threadGroups.length);
 
         final List<ThreadGroup> result = new ArrayList<>(count);
-        for(int i = 0; i < count; ++i) {
-            if(predicate.test(threadGroups[i])) {
+        for (int i = 0; i < count; ++i) {
+            if (predicate.test(threadGroups[i])) {
                 result.add(threadGroups[i]);
             }
         }

http://git-wip-us.apache.org/repos/asf/commons-lang/blob/89cbfa54/src/main/java/org/apache/commons/lang3/builder/ToStringStyle.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/commons/lang3/builder/ToStringStyle.java b/src/main/java/org/apache/commons/lang3/builder/ToStringStyle.java
index d54b19c..cf8e79a 100644
--- a/src/main/java/org/apache/commons/lang3/builder/ToStringStyle.java
+++ b/src/main/java/org/apache/commons/lang3/builder/ToStringStyle.java
@@ -2407,7 +2407,7 @@ public abstract class ToStringStyle implements Serializable {
                 throw new UnsupportedOperationException(
                         "Field names are mandatory when using JsonToStringStyle");
             }
-            if (!isFullDetail(fullDetail)){
+            if (!isFullDetail(fullDetail)) {
                 throw new UnsupportedOperationException(
                         "FullDetail must be true when using JsonToStringStyle");
             }
@@ -2423,7 +2423,7 @@ public abstract class ToStringStyle implements Serializable {
                 throw new UnsupportedOperationException(
                         "Field names are mandatory when using JsonToStringStyle");
             }
-            if (!isFullDetail(fullDetail)){
+            if (!isFullDetail(fullDetail)) {
                 throw new UnsupportedOperationException(
                         "FullDetail must be true when using JsonToStringStyle");
             }
@@ -2439,7 +2439,7 @@ public abstract class ToStringStyle implements Serializable {
                 throw new UnsupportedOperationException(
                         "Field names are mandatory when using JsonToStringStyle");
             }
-            if (!isFullDetail(fullDetail)){
+            if (!isFullDetail(fullDetail)) {
                 throw new UnsupportedOperationException(
                         "FullDetail must be true when using JsonToStringStyle");
             }
@@ -2455,7 +2455,7 @@ public abstract class ToStringStyle implements Serializable {
                 throw new UnsupportedOperationException(
                         "Field names are mandatory when using JsonToStringStyle");
             }
-            if (!isFullDetail(fullDetail)){
+            if (!isFullDetail(fullDetail)) {
                 throw new UnsupportedOperationException(
                         "FullDetail must be true when using JsonToStringStyle");
             }
@@ -2471,7 +2471,7 @@ public abstract class ToStringStyle implements Serializable {
                 throw new UnsupportedOperationException(
                         "Field names are mandatory when using JsonToStringStyle");
             }
-            if (!isFullDetail(fullDetail)){
+            if (!isFullDetail(fullDetail)) {
                 throw new UnsupportedOperationException(
                         "FullDetail must be true when using JsonToStringStyle");
             }
@@ -2487,7 +2487,7 @@ public abstract class ToStringStyle implements Serializable {
                 throw new UnsupportedOperationException(
                         "Field names are mandatory when using JsonToStringStyle");
             }
-            if (!isFullDetail(fullDetail)){
+            if (!isFullDetail(fullDetail)) {
                 throw new UnsupportedOperationException(
                         "FullDetail must be true when using JsonToStringStyle");
             }
@@ -2503,7 +2503,7 @@ public abstract class ToStringStyle implements Serializable {
                 throw new UnsupportedOperationException(
                         "Field names are mandatory when using JsonToStringStyle");
             }
-            if (!isFullDetail(fullDetail)){
+            if (!isFullDetail(fullDetail)) {
                 throw new UnsupportedOperationException(
                         "FullDetail must be true when using JsonToStringStyle");
             }
@@ -2519,7 +2519,7 @@ public abstract class ToStringStyle implements Serializable {
                 throw new UnsupportedOperationException(
                         "Field names are mandatory when using JsonToStringStyle");
             }
-            if (!isFullDetail(fullDetail)){
+            if (!isFullDetail(fullDetail)) {
                 throw new UnsupportedOperationException(
                         "FullDetail must be true when using JsonToStringStyle");
             }
@@ -2535,7 +2535,7 @@ public abstract class ToStringStyle implements Serializable {
                 throw new UnsupportedOperationException(
                         "Field names are mandatory when using JsonToStringStyle");
             }
-            if (!isFullDetail(fullDetail)){
+            if (!isFullDetail(fullDetail)) {
                 throw new UnsupportedOperationException(
                         "FullDetail must be true when using JsonToStringStyle");
             }
@@ -2551,7 +2551,7 @@ public abstract class ToStringStyle implements Serializable {
                 throw new UnsupportedOperationException(
                         "Field names are mandatory when using JsonToStringStyle");
             }
-            if (!isFullDetail(fullDetail)){
+            if (!isFullDetail(fullDetail)) {
                 throw new UnsupportedOperationException(
                         "FullDetail must be true when using JsonToStringStyle");
             }

http://git-wip-us.apache.org/repos/asf/commons-lang/blob/89cbfa54/src/main/java/org/apache/commons/lang3/concurrent/TimedSemaphore.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/commons/lang3/concurrent/TimedSemaphore.java b/src/main/java/org/apache/commons/lang3/concurrent/TimedSemaphore.java
index a3517bd..7fce712 100644
--- a/src/main/java/org/apache/commons/lang3/concurrent/TimedSemaphore.java
+++ b/src/main/java/org/apache/commons/lang3/concurrent/TimedSemaphore.java
@@ -65,7 +65,7 @@ import java.util.concurrent.TimeUnit;
  *     // Gather statistics
  *     public void run() {
  *         try {
- *             while(true) {
+ *             while (true) {
  *                 semaphore.acquire();   // limit database load
  *                 performQuery();        // issue a query
  *             }

http://git-wip-us.apache.org/repos/asf/commons-lang/blob/89cbfa54/src/main/java/org/apache/commons/lang3/exception/ContextedException.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/commons/lang3/exception/ContextedException.java b/src/main/java/org/apache/commons/lang3/exception/ContextedException.java
index 4ab95f3..02b2b22 100644
--- a/src/main/java/org/apache/commons/lang3/exception/ContextedException.java
+++ b/src/main/java/org/apache/commons/lang3/exception/ContextedException.java
@@ -231,7 +231,7 @@ public class ContextedException extends Exception implements ExceptionContext {
      * @return the message, never null
      */
     @Override
-    public String getMessage(){
+    public String getMessage() {
         return getFormattedExceptionMessage(super.getMessage());
     }
 

http://git-wip-us.apache.org/repos/asf/commons-lang/blob/89cbfa54/src/main/java/org/apache/commons/lang3/exception/ContextedRuntimeException.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/commons/lang3/exception/ContextedRuntimeException.java b/src/main/java/org/apache/commons/lang3/exception/ContextedRuntimeException.java
index 9b904fc..12db259 100644
--- a/src/main/java/org/apache/commons/lang3/exception/ContextedRuntimeException.java
+++ b/src/main/java/org/apache/commons/lang3/exception/ContextedRuntimeException.java
@@ -231,7 +231,7 @@ public class ContextedRuntimeException extends RuntimeException implements Excep
      * @return the message, never null
      */
     @Override
-    public String getMessage(){
+    public String getMessage() {
         return getFormattedExceptionMessage(super.getMessage());
     }
 

http://git-wip-us.apache.org/repos/asf/commons-lang/blob/89cbfa54/src/main/java/org/apache/commons/lang3/exception/DefaultExceptionContext.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/commons/lang3/exception/DefaultExceptionContext.java b/src/main/java/org/apache/commons/lang3/exception/DefaultExceptionContext.java
index 277c224..20a3e17 100644
--- a/src/main/java/org/apache/commons/lang3/exception/DefaultExceptionContext.java
+++ b/src/main/java/org/apache/commons/lang3/exception/DefaultExceptionContext.java
@@ -124,7 +124,7 @@ public class DefaultExceptionContext implements ExceptionContext, Serializable {
      * @return the exception message <b>with</b> context information appended, never null
      */
     @Override
-    public String getFormattedExceptionMessage(final String baseMessage){
+    public String getFormattedExceptionMessage(final String baseMessage) {
         final StringBuilder buffer = new StringBuilder(256);
         if (baseMessage != null) {
             buffer.append(baseMessage);

http://git-wip-us.apache.org/repos/asf/commons-lang/blob/89cbfa54/src/main/java/org/apache/commons/lang3/math/IEEE754rUtils.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/commons/lang3/math/IEEE754rUtils.java b/src/main/java/org/apache/commons/lang3/math/IEEE754rUtils.java
index 8cd351b..fea8504 100644
--- a/src/main/java/org/apache/commons/lang3/math/IEEE754rUtils.java
+++ b/src/main/java/org/apache/commons/lang3/math/IEEE754rUtils.java
@@ -95,10 +95,10 @@ public class IEEE754rUtils {
      * @return  the smallest of the values
      */
     public static double min(final double a, final double b) {
-        if(Double.isNaN(a)) {
+        if (Double.isNaN(a)) {
             return b;
         } else
-        if(Double.isNaN(b)) {
+        if (Double.isNaN(b)) {
             return a;
         } else {
             return Math.min(a, b);
@@ -129,10 +129,10 @@ public class IEEE754rUtils {
      * @return  the smallest of the values
      */
     public static float min(final float a, final float b) {
-        if(Float.isNaN(a)) {
+        if (Float.isNaN(a)) {
             return b;
         } else
-        if(Float.isNaN(b)) {
+        if (Float.isNaN(b)) {
             return a;
         } else {
             return Math.min(a, b);
@@ -207,10 +207,10 @@ public class IEEE754rUtils {
      * @return  the largest of the values
      */
     public static double max(final double a, final double b) {
-        if(Double.isNaN(a)) {
+        if (Double.isNaN(a)) {
             return b;
         } else
-        if(Double.isNaN(b)) {
+        if (Double.isNaN(b)) {
             return a;
         } else {
             return Math.max(a, b);
@@ -241,10 +241,10 @@ public class IEEE754rUtils {
      * @return  the largest of the values
      */
     public static float max(final float a, final float b) {
-        if(Float.isNaN(a)) {
+        if (Float.isNaN(a)) {
             return b;
         } else
-        if(Float.isNaN(b)) {
+        if (Float.isNaN(b)) {
             return a;
         } else {
             return Math.max(a, b);

http://git-wip-us.apache.org/repos/asf/commons-lang/blob/89cbfa54/src/main/java/org/apache/commons/lang3/math/NumberUtils.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/commons/lang3/math/NumberUtils.java b/src/main/java/org/apache/commons/lang3/math/NumberUtils.java
index 2522aca..e3efc96 100644
--- a/src/main/java/org/apache/commons/lang3/math/NumberUtils.java
+++ b/src/main/java/org/apache/commons/lang3/math/NumberUtils.java
@@ -122,7 +122,7 @@ public class NumberUtils {
      * @since 2.1
      */
     public static int toInt(final String str, final int defaultValue) {
-        if(str == null) {
+        if (str == null) {
             return defaultValue;
         }
         try {
@@ -366,7 +366,7 @@ public class NumberUtils {
      * @since 2.5
      */
     public static byte toByte(final String str, final byte defaultValue) {
-        if(str == null) {
+        if (str == null) {
             return defaultValue;
         }
         try {
@@ -415,7 +415,7 @@ public class NumberUtils {
      * @since 2.5
      */
     public static short toShort(final String str, final short defaultValue) {
-        if(str == null) {
+        if (str == null) {
             return defaultValue;
         }
         try {
@@ -585,33 +585,33 @@ public class NumberUtils {
     //                  BigDecimal, BigInteger and Byte
     // useful methods:
     // Byte.decode(String)
-    // Byte.valueOf(String,int radix)
+    // Byte.valueOf(String, int radix)
     // Byte.valueOf(String)
     // Double.valueOf(String)
     // Float.valueOf(String)
     // Float.valueOf(String)
-    // Integer.valueOf(String,int radix)
+    // Integer.valueOf(String, int radix)
     // Integer.valueOf(String)
     // Integer.decode(String)
     // Integer.getInteger(String)
-    // Integer.getInteger(String,int val)
-    // Integer.getInteger(String,Integer val)
+    // Integer.getInteger(String, int val)
+    // Integer.getInteger(String, Integer val)
     // Integer.valueOf(String)
     // Double.valueOf(String)
     // new Byte(String)
     // Long.valueOf(String)
     // Long.getLong(String)
-    // Long.getLong(String,int)
-    // Long.getLong(String,Integer)
-    // Long.valueOf(String,int)
+    // Long.getLong(String, int)
+    // Long.getLong(String, Integer)
+    // Long.valueOf(String, int)
     // Long.valueOf(String)
     // Short.valueOf(String)
     // Short.decode(String)
-    // Short.valueOf(String,int)
+    // Short.valueOf(String, int)
     // Short.valueOf(String)
     // new BigDecimal(String)
     // new BigInteger(String)
-    // new BigInteger(String,int radix)
+    // new BigInteger(String, int radix)
     // Possible inputs:
     // 45 45.5 45E7 4.5E7 Hex Oct Binary xxxF xxxD xxxf xxxd
     // plus minus everything. Prolly more. A lot are not separable.
@@ -624,7 +624,7 @@ public class NumberUtils {
      * prefix is more than 8 - or BigInteger if there are more than 16 digits.
      * </p>
      * <p>Then, the value is examined for a type qualifier on the end, i.e. one of
-     * <code>'f','F','d','D','l','L'</code>.  If it is found, it starts
+     * <code>'f', 'F', 'd', 'D', 'l', 'L'</code>.  If it is found, it starts
      * trying to create successively larger types from the type specified
      * until one is found that can represent the value.</p>
      *
@@ -657,7 +657,7 @@ public class NumberUtils {
         // Need to deal with all possible hex prefixes here
         final String[] hex_prefixes = {"0x", "0X", "-0x", "-0X", "#", "-#"};
         int pfxLen = 0;
-        for(final String pfx : hex_prefixes) {
+        for (final String pfx : hex_prefixes) {
             if (str.startsWith(pfx)) {
                 pfxLen += pfx.length();
                 break;
@@ -665,7 +665,7 @@ public class NumberUtils {
         }
         if (pfxLen > 0) { // we have a hex number
             char firstSigDigit = 0; // strip leading zeroes
-            for(int i = pfxLen; i < str.length(); i++) {
+            for (int i = pfxLen; i < str.length(); i++) {
                 firstSigDigit = str.charAt(i);
                 if (firstSigDigit == '0') { // count leading zeroes
                     pfxLen++;

http://git-wip-us.apache.org/repos/asf/commons-lang/blob/89cbfa54/src/main/java/org/apache/commons/lang3/package-info.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/commons/lang3/package-info.java b/src/main/java/org/apache/commons/lang3/package-info.java
index b9f22ba..9a476fe 100644
--- a/src/main/java/org/apache/commons/lang3/package-info.java
+++ b/src/main/java/org/apache/commons/lang3/package-info.java
@@ -61,7 +61,7 @@
  * <p>SystemUtils is a simple little class which makes it easy to find out information about which platform you are on.
  * For some, this is a necessary evil. It was never something I expected to use myself until I was trying to ensure that Commons Lang itself compiled under JDK 1.2.
  * Having pushed out a few JDK 1.3 bits that had slipped in (<code>Collections.EMPTY_MAP</code> is a classic offender), I then found that one of the Unit Tests was dying mysteriously under JDK 1.2, but ran fine under JDK 1.3.
- * There was no obvious solution and I needed to move onwards, so the simple solution was to wrap that particular test in a <code>if(SystemUtils.isJavaVersionAtLeast(1.3f)) {</code>, make a note and move on.</p>
+ * There was no obvious solution and I needed to move onwards, so the simple solution was to wrap that particular test in a <code>if (SystemUtils.isJavaVersionAtLeast(1.3f)) {</code>, make a note and move on.</p>
  *
  * <p>The {@link org.apache.commons.lang3.CharEncoding} class is also used to interact with the Java environment and may be used to see which character encodings are supported in a particular environment. </p>
  *

http://git-wip-us.apache.org/repos/asf/commons-lang/blob/89cbfa54/src/main/java/org/apache/commons/lang3/reflect/ConstructorUtils.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/commons/lang3/reflect/ConstructorUtils.java b/src/main/java/org/apache/commons/lang3/reflect/ConstructorUtils.java
index b6fb70d..f95af54 100644
--- a/src/main/java/org/apache/commons/lang3/reflect/ConstructorUtils.java
+++ b/src/main/java/org/apache/commons/lang3/reflect/ConstructorUtils.java
@@ -271,7 +271,7 @@ public class ConstructorUtils {
                         // temporary variable for annotation, see comment above (1)
                         @SuppressWarnings("unchecked")
                         final
-                        Constructor<T> constructor = (Constructor<T>)ctor;
+                        Constructor<T> constructor = (Constructor<T>) ctor;
                         result = constructor;
                     }
                 }

http://git-wip-us.apache.org/repos/asf/commons-lang/blob/89cbfa54/src/main/java/org/apache/commons/lang3/reflect/MethodUtils.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/commons/lang3/reflect/MethodUtils.java b/src/main/java/org/apache/commons/lang3/reflect/MethodUtils.java
index 5874f59..a9a8031 100644
--- a/src/main/java/org/apache/commons/lang3/reflect/MethodUtils.java
+++ b/src/main/java/org/apache/commons/lang3/reflect/MethodUtils.java
@@ -77,7 +77,7 @@ public class MethodUtils {
      * <p>This method delegates the method search to {@link #getMatchingAccessibleMethod(Class, String, Class[])}.</p>
      *
      * <p>This is a convenient wrapper for
-     * {@link #invokeMethod(Object object,String methodName, Object[] args, Class[] parameterTypes)}.
+     * {@link #invokeMethod(Object object, String methodName, Object[] args, Class[] parameterTypes)}.
      * </p>
      *
      * @param object invoke method on this object
@@ -99,7 +99,7 @@ public class MethodUtils {
      * <p>Invokes a named method without parameters.</p>
      *
      * <p>This is a convenient wrapper for
-     * {@link #invokeMethod(Object object,boolean forceAccess,String methodName, Object[] args, Class[] parameterTypes)}.
+     * {@link #invokeMethod(Object object, boolean forceAccess, String methodName, Object[] args, Class[] parameterTypes)}.
      * </p>
      *
      * @param object invoke method on this object
@@ -128,7 +128,7 @@ public class MethodUtils {
      * would match a {@code boolean} primitive.</p>
      *
      * <p>This is a convenient wrapper for
-     * {@link #invokeMethod(Object object,String methodName, Object[] args, Class[] parameterTypes)}.
+     * {@link #invokeMethod(Object object, String methodName, Object[] args, Class[] parameterTypes)}.
      * </p>
      *
      * @param object invoke method on this object
@@ -156,7 +156,7 @@ public class MethodUtils {
      * would match a {@code boolean} primitive.</p>
      *
      * <p>This is a convenient wrapper for
-     * {@link #invokeMethod(Object object,boolean forceAccess,String methodName, Object[] args, Class[] parameterTypes)}.
+     * {@link #invokeMethod(Object object, boolean forceAccess, String methodName, Object[] args, Class[] parameterTypes)}.
      * </p>
      *
      * @param object invoke method on this object
@@ -261,7 +261,7 @@ public class MethodUtils {
      * types.</p>
      *
      * <p>This uses reflection to invoke the method obtained from a call to
-     * {@link #getAccessibleMethod}(Class,String,Class[])}.</p>
+     * {@link #getAccessibleMethod}(Class, String, Class[])}.</p>
      *
      * @param object invoke method on this object
      * @param methodName get method with this name
@@ -284,7 +284,7 @@ public class MethodUtils {
      * <p>Invokes a method with no parameters.</p>
      *
      * <p>This uses reflection to invoke the method obtained from a call to
-     * {@link #getAccessibleMethod}(Class,String,Class[])}.</p>
+     * {@link #getAccessibleMethod}(Class, String, Class[])}.</p>
      *
      * @param object invoke method on this object
      * @param methodName get method with this name
@@ -310,7 +310,7 @@ public class MethodUtils {
      * types given.</p>
      *
      * <p>This uses reflection to invoke the method obtained from a call to
-     * {@link #getAccessibleMethod(Class,String,Class[])}.</p>
+     * {@link #getAccessibleMethod(Class, String, Class[])}.</p>
      *
      * @param object invoke method on this object
      * @param methodName get method with this name
@@ -482,7 +482,7 @@ public class MethodUtils {
         // Copy the variadic arguments into the varargs array.
         System.arraycopy(args, methodParameterTypes.length - 1, varArgsArray, 0, varArgLength);
 
-        if(varArgComponentType.isPrimitive()) {
+        if (varArgComponentType.isPrimitive()) {
             // unbox from wrapper type to primitive type
             varArgsArray = ArrayUtils.toPrimitive(varArgsArray);
         }

http://git-wip-us.apache.org/repos/asf/commons-lang/blob/89cbfa54/src/main/java/org/apache/commons/lang3/reflect/TypeUtils.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/commons/lang3/reflect/TypeUtils.java b/src/main/java/org/apache/commons/lang3/reflect/TypeUtils.java
index 0ec0fb6..6402303 100644
--- a/src/main/java/org/apache/commons/lang3/reflect/TypeUtils.java
+++ b/src/main/java/org/apache/commons/lang3/reflect/TypeUtils.java
@@ -737,7 +737,7 @@ public class TypeUtils {
     /**
      * <p>Retrieves all the type arguments for this parameterized type
      * including owner hierarchy arguments such as
-     * {@code Outer<K,V>.Inner<T>.DeepInner<E>} .
+     * {@code Outer<K, V>.Inner<T>.DeepInner<E>} .
      * The arguments are returned in a
      * {@link Map} specifying the argument type for each {@link TypeVariable}.
      * </p>

http://git-wip-us.apache.org/repos/asf/commons-lang/blob/89cbfa54/src/main/java/org/apache/commons/lang3/text/ExtendedMessageFormat.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/commons/lang3/text/ExtendedMessageFormat.java b/src/main/java/org/apache/commons/lang3/text/ExtendedMessageFormat.java
index 573b013..76f65e6 100644
--- a/src/main/java/org/apache/commons/lang3/text/ExtendedMessageFormat.java
+++ b/src/main/java/org/apache/commons/lang3/text/ExtendedMessageFormat.java
@@ -276,7 +276,7 @@ public class ExtendedMessageFormat extends MessageFormat {
         if (ObjectUtils.notEqual(getClass(), obj.getClass())) {
           return false;
         }
-        final ExtendedMessageFormat rhs = (ExtendedMessageFormat)obj;
+        final ExtendedMessageFormat rhs = (ExtendedMessageFormat) obj;
         if (ObjectUtils.notEqual(toPattern, rhs.toPattern)) {
             return false;
         }
@@ -478,7 +478,7 @@ public class ExtendedMessageFormat extends MessageFormat {
             "Quoted string must start with quote character";
 
         // handle quote character at the beginning of the string
-        if(appendTo != null) {
+        if (appendTo != null) {
             appendTo.append(QUOTE);
         }
         next(pos);

http://git-wip-us.apache.org/repos/asf/commons-lang/blob/89cbfa54/src/main/java/org/apache/commons/lang3/text/StrBuilder.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/commons/lang3/text/StrBuilder.java b/src/main/java/org/apache/commons/lang3/text/StrBuilder.java
index d07279d..262a9d7 100644
--- a/src/main/java/org/apache/commons/lang3/text/StrBuilder.java
+++ b/src/main/java/org/apache/commons/lang3/text/StrBuilder.java
@@ -1348,11 +1348,11 @@ public class StrBuilder implements CharSequence, Appendable, Serializable, Build
      * This method is for example useful for constructing queries
      * <pre>
      * StrBuilder whereClause = new StrBuilder();
-     * if(searchCommand.getPriority() != null) {
+     * if (searchCommand.getPriority() != null) {
      *  whereClause.appendSeparator(" and", " where");
      *  whereClause.append(" priority = ?")
      * }
-     * if(searchCommand.getComponent() != null) {
+     * if (searchCommand.getComponent() != null) {
      *  whereClause.appendSeparator(" and", " where");
      *  whereClause.append(" component = ?")
      * }
@@ -2145,7 +2145,7 @@ public class StrBuilder implements CharSequence, Appendable, Serializable, Build
 
         final int half = size / 2;
         final char[] buf = buffer;
-        for (int leftIdx = 0, rightIdx = size - 1; leftIdx < half; leftIdx++,rightIdx--) {
+        for (int leftIdx = 0, rightIdx = size - 1; leftIdx < half; leftIdx++, rightIdx--) {
             final char swap = buf[leftIdx];
             buf[leftIdx] = buf[rightIdx];
             buf[rightIdx] = swap;
@@ -2230,7 +2230,7 @@ public class StrBuilder implements CharSequence, Appendable, Serializable, Build
             return false;
         }
         int pos = size - len;
-        for (int i = 0; i < len; i++,pos++) {
+        for (int i = 0; i < len; i++, pos++) {
             if (buffer[pos] != str.charAt(i)) {
                 return false;
             }

http://git-wip-us.apache.org/repos/asf/commons-lang/blob/89cbfa54/src/main/java/org/apache/commons/lang3/text/StrSubstitutor.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/commons/lang3/text/StrSubstitutor.java b/src/main/java/org/apache/commons/lang3/text/StrSubstitutor.java
index d02418e..b8c885e 100644
--- a/src/main/java/org/apache/commons/lang3/text/StrSubstitutor.java
+++ b/src/main/java/org/apache/commons/lang3/text/StrSubstitutor.java
@@ -218,10 +218,10 @@ public class StrSubstitutor {
         if (valueProperties == null) {
             return source.toString();
         }
-        final Map<String,String> valueMap = new HashMap<>();
+        final Map<String, String> valueMap = new HashMap<>();
         final Enumeration<?> propNames = valueProperties.propertyNames();
         while (propNames.hasMoreElements()) {
-            final String propName = (String)propNames.nextElement();
+            final String propName = (String) propNames.nextElement();
             final String propValue = valueProperties.getProperty(propName);
             valueMap.put(propName, propValue);
         }

http://git-wip-us.apache.org/repos/asf/commons-lang/blob/89cbfa54/src/main/java/org/apache/commons/lang3/text/WordUtils.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/commons/lang3/text/WordUtils.java b/src/main/java/org/apache/commons/lang3/text/WordUtils.java
index 3787f33..f7c420c 100644
--- a/src/main/java/org/apache/commons/lang3/text/WordUtils.java
+++ b/src/main/java/org/apache/commons/lang3/text/WordUtils.java
@@ -292,7 +292,7 @@ public class WordUtils {
         while (offset < inputLineLength) {
             int spaceToWrapAt = -1;
             Matcher matcher = patternToWrapOn.matcher(
-                str.substring(offset, Math.min((int)Math.min(Integer.MAX_VALUE, offset + wrapLength + 1L), inputLineLength)));
+                str.substring(offset, Math.min((int) Math.min(Integer.MAX_VALUE, offset + wrapLength + 1L), inputLineLength)));
             if (matcher.find()) {
                 if (matcher.start() == 0) {
                     offset += matcher.end();
@@ -302,11 +302,11 @@ public class WordUtils {
             }
 
             // only last line without leading spaces is left
-            if(inputLineLength - offset <= wrapLength) {
+            if (inputLineLength - offset <= wrapLength) {
                 break;
             }
 
-            while(matcher.find()){
+            while (matcher.find()) {
                 spaceToWrapAt = matcher.start() + offset;
             }
 

http://git-wip-us.apache.org/repos/asf/commons-lang/blob/89cbfa54/src/main/java/org/apache/commons/lang3/text/translate/AggregateTranslator.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/commons/lang3/text/translate/AggregateTranslator.java b/src/main/java/org/apache/commons/lang3/text/translate/AggregateTranslator.java
index 86b22bb..a800f46 100644
--- a/src/main/java/org/apache/commons/lang3/text/translate/AggregateTranslator.java
+++ b/src/main/java/org/apache/commons/lang3/text/translate/AggregateTranslator.java
@@ -53,7 +53,7 @@ public class AggregateTranslator extends CharSequenceTranslator {
     public int translate(final CharSequence input, final int index, final Writer out) throws IOException {
         for (final CharSequenceTranslator translator : translators) {
             final int consumed = translator.translate(input, index, out);
-            if(consumed != 0) {
+            if (consumed != 0) {
                 return consumed;
             }
         }

http://git-wip-us.apache.org/repos/asf/commons-lang/blob/89cbfa54/src/main/java/org/apache/commons/lang3/text/translate/EntityArrays.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/commons/lang3/text/translate/EntityArrays.java b/src/main/java/org/apache/commons/lang3/text/translate/EntityArrays.java
index d4357f6..864779e 100644
--- a/src/main/java/org/apache/commons/lang3/text/translate/EntityArrays.java
+++ b/src/main/java/org/apache/commons/lang3/text/translate/EntityArrays.java
@@ -163,15 +163,15 @@ public class EntityArrays {
         // <!-- Greek -->
         {"\u0391", "&Alpha;"}, // greek capital letter alpha, U+0391 -->
         {"\u0392", "&Beta;"}, // greek capital letter beta, U+0392 -->
-        {"\u0393", "&Gamma;"}, // greek capital letter gamma,U+0393 ISOgrk3 -->
-        {"\u0394", "&Delta;"}, // greek capital letter delta,U+0394 ISOgrk3 -->
+        {"\u0393", "&Gamma;"}, // greek capital letter gamma, U+0393 ISOgrk3 -->
+        {"\u0394", "&Delta;"}, // greek capital letter delta, U+0394 ISOgrk3 -->
         {"\u0395", "&Epsilon;"}, // greek capital letter epsilon, U+0395 -->
         {"\u0396", "&Zeta;"}, // greek capital letter zeta, U+0396 -->
         {"\u0397", "&Eta;"}, // greek capital letter eta, U+0397 -->
-        {"\u0398", "&Theta;"}, // greek capital letter theta,U+0398 ISOgrk3 -->
+        {"\u0398", "&Theta;"}, // greek capital letter theta, U+0398 ISOgrk3 -->
         {"\u0399", "&Iota;"}, // greek capital letter iota, U+0399 -->
         {"\u039A", "&Kappa;"}, // greek capital letter kappa, U+039A -->
-        {"\u039B", "&Lambda;"}, // greek capital letter lambda,U+039B ISOgrk3 -->
+        {"\u039B", "&Lambda;"}, // greek capital letter lambda, U+039B ISOgrk3 -->
         {"\u039C", "&Mu;"}, // greek capital letter mu, U+039C -->
         {"\u039D", "&Nu;"}, // greek capital letter nu, U+039D -->
         {"\u039E", "&Xi;"}, // greek capital letter xi, U+039E ISOgrk3 -->
@@ -179,56 +179,56 @@ public class EntityArrays {
         {"\u03A0", "&Pi;"}, // greek capital letter pi, U+03A0 ISOgrk3 -->
         {"\u03A1", "&Rho;"}, // greek capital letter rho, U+03A1 -->
         // <!-- there is no Sigmaf, and no U+03A2 character either -->
-        {"\u03A3", "&Sigma;"}, // greek capital letter sigma,U+03A3 ISOgrk3 -->
+        {"\u03A3", "&Sigma;"}, // greek capital letter sigma, U+03A3 ISOgrk3 -->
         {"\u03A4", "&Tau;"}, // greek capital letter tau, U+03A4 -->
-        {"\u03A5", "&Upsilon;"}, // greek capital letter upsilon,U+03A5 ISOgrk3 -->
-        {"\u03A6", "&Phi;"}, // greek capital letter phi,U+03A6 ISOgrk3 -->
+        {"\u03A5", "&Upsilon;"}, // greek capital letter upsilon, U+03A5 ISOgrk3 -->
+        {"\u03A6", "&Phi;"}, // greek capital letter phi, U+03A6 ISOgrk3 -->
         {"\u03A7", "&Chi;"}, // greek capital letter chi, U+03A7 -->
-        {"\u03A8", "&Psi;"}, // greek capital letter psi,U+03A8 ISOgrk3 -->
-        {"\u03A9", "&Omega;"}, // greek capital letter omega,U+03A9 ISOgrk3 -->
-        {"\u03B1", "&alpha;"}, // greek small letter alpha,U+03B1 ISOgrk3 -->
+        {"\u03A8", "&Psi;"}, // greek capital letter psi, U+03A8 ISOgrk3 -->
+        {"\u03A9", "&Omega;"}, // greek capital letter omega, U+03A9 ISOgrk3 -->
+        {"\u03B1", "&alpha;"}, // greek small letter alpha, U+03B1 ISOgrk3 -->
         {"\u03B2", "&beta;"}, // greek small letter beta, U+03B2 ISOgrk3 -->
-        {"\u03B3", "&gamma;"}, // greek small letter gamma,U+03B3 ISOgrk3 -->
-        {"\u03B4", "&delta;"}, // greek small letter delta,U+03B4 ISOgrk3 -->
-        {"\u03B5", "&epsilon;"}, // greek small letter epsilon,U+03B5 ISOgrk3 -->
+        {"\u03B3", "&gamma;"}, // greek small letter gamma, U+03B3 ISOgrk3 -->
+        {"\u03B4", "&delta;"}, // greek small letter delta, U+03B4 ISOgrk3 -->
+        {"\u03B5", "&epsilon;"}, // greek small letter epsilon, U+03B5 ISOgrk3 -->
         {"\u03B6", "&zeta;"}, // greek small letter zeta, U+03B6 ISOgrk3 -->
         {"\u03B7", "&eta;"}, // greek small letter eta, U+03B7 ISOgrk3 -->
-        {"\u03B8", "&theta;"}, // greek small letter theta,U+03B8 ISOgrk3 -->
+        {"\u03B8", "&theta;"}, // greek small letter theta, U+03B8 ISOgrk3 -->
         {"\u03B9", "&iota;"}, // greek small letter iota, U+03B9 ISOgrk3 -->
-        {"\u03BA", "&kappa;"}, // greek small letter kappa,U+03BA ISOgrk3 -->
-        {"\u03BB", "&lambda;"}, // greek small letter lambda,U+03BB ISOgrk3 -->
+        {"\u03BA", "&kappa;"}, // greek small letter kappa, U+03BA ISOgrk3 -->
+        {"\u03BB", "&lambda;"}, // greek small letter lambda, U+03BB ISOgrk3 -->
         {"\u03BC", "&mu;"}, // greek small letter mu, U+03BC ISOgrk3 -->
         {"\u03BD", "&nu;"}, // greek small letter nu, U+03BD ISOgrk3 -->
         {"\u03BE", "&xi;"}, // greek small letter xi, U+03BE ISOgrk3 -->
         {"\u03BF", "&omicron;"}, // greek small letter omicron, U+03BF NEW -->
         {"\u03C0", "&pi;"}, // greek small letter pi, U+03C0 ISOgrk3 -->
         {"\u03C1", "&rho;"}, // greek small letter rho, U+03C1 ISOgrk3 -->
-        {"\u03C2", "&sigmaf;"}, // greek small letter final sigma,U+03C2 ISOgrk3 -->
-        {"\u03C3", "&sigma;"}, // greek small letter sigma,U+03C3 ISOgrk3 -->
+        {"\u03C2", "&sigmaf;"}, // greek small letter final sigma, U+03C2 ISOgrk3 -->
+        {"\u03C3", "&sigma;"}, // greek small letter sigma, U+03C3 ISOgrk3 -->
         {"\u03C4", "&tau;"}, // greek small letter tau, U+03C4 ISOgrk3 -->
-        {"\u03C5", "&upsilon;"}, // greek small letter upsilon,U+03C5 ISOgrk3 -->
+        {"\u03C5", "&upsilon;"}, // greek small letter upsilon, U+03C5 ISOgrk3 -->
         {"\u03C6", "&phi;"}, // greek small letter phi, U+03C6 ISOgrk3 -->
         {"\u03C7", "&chi;"}, // greek small letter chi, U+03C7 ISOgrk3 -->
         {"\u03C8", "&psi;"}, // greek small letter psi, U+03C8 ISOgrk3 -->
-        {"\u03C9", "&omega;"}, // greek small letter omega,U+03C9 ISOgrk3 -->
-        {"\u03D1", "&thetasym;"}, // greek small letter theta symbol,U+03D1 NEW -->
-        {"\u03D2", "&upsih;"}, // greek upsilon with hook symbol,U+03D2 NEW -->
+        {"\u03C9", "&omega;"}, // greek small letter omega, U+03C9 ISOgrk3 -->
+        {"\u03D1", "&thetasym;"}, // greek small letter theta symbol, U+03D1 NEW -->
+        {"\u03D2", "&upsih;"}, // greek upsilon with hook symbol, U+03D2 NEW -->
         {"\u03D6", "&piv;"}, // greek pi symbol, U+03D6 ISOgrk3 -->
         // <!-- General Punctuation -->
-        {"\u2022", "&bull;"}, // bullet = black small circle,U+2022 ISOpub -->
+        {"\u2022", "&bull;"}, // bullet = black small circle, U+2022 ISOpub -->
         // <!-- bullet is NOT the same as bullet operator, U+2219 -->
-        {"\u2026", "&hellip;"}, // horizontal ellipsis = three dot leader,U+2026 ISOpub -->
+        {"\u2026", "&hellip;"}, // horizontal ellipsis = three dot leader, U+2026 ISOpub -->
         {"\u2032", "&prime;"}, // prime = minutes = feet, U+2032 ISOtech -->
-        {"\u2033", "&Prime;"}, // double prime = seconds = inches,U+2033 ISOtech -->
-        {"\u203E", "&oline;"}, // overline = spacing overscore,U+203E NEW -->
+        {"\u2033", "&Prime;"}, // double prime = seconds = inches, U+2033 ISOtech -->
+        {"\u203E", "&oline;"}, // overline = spacing overscore, U+203E NEW -->
         {"\u2044", "&frasl;"}, // fraction slash, U+2044 NEW -->
         // <!-- Letterlike Symbols -->
         {"\u2118", "&weierp;"}, // script capital P = power set= Weierstrass p, U+2118 ISOamso -->
-        {"\u2111", "&image;"}, // blackletter capital I = imaginary part,U+2111 ISOamso -->
-        {"\u211C", "&real;"}, // blackletter capital R = real part symbol,U+211C ISOamso -->
+        {"\u2111", "&image;"}, // blackletter capital I = imaginary part, U+2111 ISOamso -->
+        {"\u211C", "&real;"}, // blackletter capital R = real part symbol, U+211C ISOamso -->
         {"\u2122", "&trade;"}, // trade mark sign, U+2122 ISOnum -->
-        {"\u2135", "&alefsym;"}, // alef symbol = first transfinite cardinal,U+2135 NEW -->
-        // <!-- alef symbol is NOT the same as hebrew letter alef,U+05D0 although the
+        {"\u2135", "&alefsym;"}, // alef symbol = first transfinite cardinal, U+2135 NEW -->
+        // <!-- alef symbol is NOT the same as hebrew letter alef, U+05D0 although the
         // same glyph could be used to depict both characters -->
         // <!-- Arrows -->
         {"\u2190", "&larr;"}, // leftwards arrow, U+2190 ISOnum -->
@@ -242,23 +242,23 @@ public class EntityArrays {
         // arrow but also does not have any other character for that function.
         // So ? lArr canbe used for 'is implied by' as ISOtech suggests -->
         {"\u21D1", "&uArr;"}, // upwards double arrow, U+21D1 ISOamsa -->
-        {"\u21D2", "&rArr;"}, // rightwards double arrow,U+21D2 ISOtech -->
+        {"\u21D2", "&rArr;"}, // rightwards double arrow, U+21D2 ISOtech -->
         // <!-- ISO 10646 does not say this is the 'implies' character but does not
         // have another character with this function so ?rArr can be used for
         // 'implies' as ISOtech suggests -->
         {"\u21D3", "&dArr;"}, // downwards double arrow, U+21D3 ISOamsa -->
-        {"\u21D4", "&hArr;"}, // left right double arrow,U+21D4 ISOamsa -->
+        {"\u21D4", "&hArr;"}, // left right double arrow, U+21D4 ISOamsa -->
         // <!-- Mathematical Operators -->
         {"\u2200", "&forall;"}, // for all, U+2200 ISOtech -->
         {"\u2202", "&part;"}, // partial differential, U+2202 ISOtech -->
         {"\u2203", "&exist;"}, // there exists, U+2203 ISOtech -->
-        {"\u2205", "&empty;"}, // empty set = null set = diameter,U+2205 ISOamso -->
-        {"\u2207", "&nabla;"}, // nabla = backward difference,U+2207 ISOtech -->
+        {"\u2205", "&empty;"}, // empty set = null set = diameter, U+2205 ISOamso -->
+        {"\u2207", "&nabla;"}, // nabla = backward difference, U+2207 ISOtech -->
         {"\u2208", "&isin;"}, // element of, U+2208 ISOtech -->
         {"\u2209", "&notin;"}, // not an element of, U+2209 ISOtech -->
         {"\u220B", "&ni;"}, // contains as member, U+220B ISOtech -->
         // <!-- should there be a more memorable name than 'ni'? -->
-        {"\u220F", "&prod;"}, // n-ary product = product sign,U+220F ISOamsb -->
+        {"\u220F", "&prod;"}, // n-ary product = product sign, U+220F ISOamsb -->
         // <!-- prod is NOT the same character as U+03A0 'greek capital letter pi'
         // though the same glyph might be used for both -->
         {"\u2211", "&sum;"}, // n-ary summation, U+2211 ISOamsb -->
@@ -266,7 +266,7 @@ public class EntityArrays {
         // though the same glyph might be used for both -->
         {"\u2212", "&minus;"}, // minus sign, U+2212 ISOtech -->
         {"\u2217", "&lowast;"}, // asterisk operator, U+2217 ISOtech -->
-        {"\u221A", "&radic;"}, // square root = radical sign,U+221A ISOtech -->
+        {"\u221A", "&radic;"}, // square root = radical sign, U+221A ISOtech -->
         {"\u221D", "&prop;"}, // proportional to, U+221D ISOtech -->
         {"\u221E", "&infin;"}, // infinity, U+221E ISOtech -->
         {"\u2220", "&ang;"}, // angle, U+2220 ISOamso -->
@@ -276,15 +276,15 @@ public class EntityArrays {
         {"\u222A", "&cup;"}, // union = cup, U+222A ISOtech -->
         {"\u222B", "&int;"}, // integral, U+222B ISOtech -->
         {"\u2234", "&there4;"}, // therefore, U+2234 ISOtech -->
-        {"\u223C", "&sim;"}, // tilde operator = varies with = similar to,U+223C ISOtech -->
-        // <!-- tilde operator is NOT the same character as the tilde, U+007E,although
+        {"\u223C", "&sim;"}, // tilde operator = varies with = similar to, U+223C ISOtech -->
+        // <!-- tilde operator is NOT the same character as the tilde, U+007E, although
         // the same glyph might be used to represent both -->
         {"\u2245", "&cong;"}, // approximately equal to, U+2245 ISOtech -->
-        {"\u2248", "&asymp;"}, // almost equal to = asymptotic to,U+2248 ISOamsr -->
+        {"\u2248", "&asymp;"}, // almost equal to = asymptotic to, U+2248 ISOamsr -->
         {"\u2260", "&ne;"}, // not equal to, U+2260 ISOtech -->
         {"\u2261", "&equiv;"}, // identical to, U+2261 ISOtech -->
         {"\u2264", "&le;"}, // less-than or equal to, U+2264 ISOtech -->
-        {"\u2265", "&ge;"}, // greater-than or equal to,U+2265 ISOtech -->
+        {"\u2265", "&ge;"}, // greater-than or equal to, U+2265 ISOtech -->
         {"\u2282", "&sub;"}, // subset of, U+2282 ISOtech -->
         {"\u2283", "&sup;"}, // superset of, U+2283 ISOtech -->
         // <!-- note that nsup, 'not a superset of, U+2283' is not covered by the
@@ -292,21 +292,21 @@ public class EntityArrays {
         // It is in ISOamsn -->,
         {"\u2284", "&nsub;"}, // not a subset of, U+2284 ISOamsn -->
         {"\u2286", "&sube;"}, // subset of or equal to, U+2286 ISOtech -->
-        {"\u2287", "&supe;"}, // superset of or equal to,U+2287 ISOtech -->
-        {"\u2295", "&oplus;"}, // circled plus = direct sum,U+2295 ISOamsb -->
-        {"\u2297", "&otimes;"}, // circled times = vector product,U+2297 ISOamsb -->
-        {"\u22A5", "&perp;"}, // up tack = orthogonal to = perpendicular,U+22A5 ISOtech -->
+        {"\u2287", "&supe;"}, // superset of or equal to, U+2287 ISOtech -->
+        {"\u2295", "&oplus;"}, // circled plus = direct sum, U+2295 ISOamsb -->
+        {"\u2297", "&otimes;"}, // circled times = vector product, U+2297 ISOamsb -->
+        {"\u22A5", "&perp;"}, // up tack = orthogonal to = perpendicular, U+22A5 ISOtech -->
         {"\u22C5", "&sdot;"}, // dot operator, U+22C5 ISOamsb -->
         // <!-- dot operator is NOT the same character as U+00B7 middle dot -->
         // <!-- Miscellaneous Technical -->
-        {"\u2308", "&lceil;"}, // left ceiling = apl upstile,U+2308 ISOamsc -->
+        {"\u2308", "&lceil;"}, // left ceiling = apl upstile, U+2308 ISOamsc -->
         {"\u2309", "&rceil;"}, // right ceiling, U+2309 ISOamsc -->
-        {"\u230A", "&lfloor;"}, // left floor = apl downstile,U+230A ISOamsc -->
+        {"\u230A", "&lfloor;"}, // left floor = apl downstile, U+230A ISOamsc -->
         {"\u230B", "&rfloor;"}, // right floor, U+230B ISOamsc -->
-        {"\u2329", "&lang;"}, // left-pointing angle bracket = bra,U+2329 ISOtech -->
+        {"\u2329", "&lang;"}, // left-pointing angle bracket = bra, U+2329 ISOtech -->
         // <!-- lang is NOT the same character as U+003C 'less than' or U+2039 'single left-pointing angle quotation
         // mark' -->
-        {"\u232A", "&rang;"}, // right-pointing angle bracket = ket,U+232A ISOtech -->
+        {"\u232A", "&rang;"}, // right-pointing angle bracket = ket, U+232A ISOtech -->
         // <!-- rang is NOT the same character as U+003E 'greater than' or U+203A
         // 'single right-pointing angle quotation mark' -->
         // <!-- Geometric Shapes -->
@@ -314,42 +314,42 @@ public class EntityArrays {
         // <!-- Miscellaneous Symbols -->
         {"\u2660", "&spades;"}, // black spade suit, U+2660 ISOpub -->
         // <!-- black here seems to mean filled as opposed to hollow -->
-        {"\u2663", "&clubs;"}, // black club suit = shamrock,U+2663 ISOpub -->
-        {"\u2665", "&hearts;"}, // black heart suit = valentine,U+2665 ISOpub -->
+        {"\u2663", "&clubs;"}, // black club suit = shamrock, U+2663 ISOpub -->
+        {"\u2665", "&hearts;"}, // black heart suit = valentine, U+2665 ISOpub -->
         {"\u2666", "&diams;"}, // black diamond suit, U+2666 ISOpub -->
 
         // <!-- Latin Extended-A -->
-        {"\u0152", "&OElig;"}, // -- latin capital ligature OE,U+0152 ISOlat2 -->
+        {"\u0152", "&OElig;"}, // -- latin capital ligature OE, U+0152 ISOlat2 -->
         {"\u0153", "&oelig;"}, // -- latin small ligature oe, U+0153 ISOlat2 -->
         // <!-- ligature is a misnomer, this is a separate character in some languages -->
-        {"\u0160", "&Scaron;"}, // -- latin capital letter S with caron,U+0160 ISOlat2 -->
-        {"\u0161", "&scaron;"}, // -- latin small letter s with caron,U+0161 ISOlat2 -->
-        {"\u0178", "&Yuml;"}, // -- latin capital letter Y with diaeresis,U+0178 ISOlat2 -->
+        {"\u0160", "&Scaron;"}, // -- latin capital letter S with caron, U+0160 ISOlat2 -->
+        {"\u0161", "&scaron;"}, // -- latin small letter s with caron, U+0161 ISOlat2 -->
+        {"\u0178", "&Yuml;"}, // -- latin capital letter Y with diaeresis, U+0178 ISOlat2 -->
         // <!-- Spacing Modifier Letters -->
-        {"\u02C6", "&circ;"}, // -- modifier letter circumflex accent,U+02C6 ISOpub -->
+        {"\u02C6", "&circ;"}, // -- modifier letter circumflex accent, U+02C6 ISOpub -->
         {"\u02DC", "&tilde;"}, // small tilde, U+02DC ISOdia -->
         // <!-- General Punctuation -->
         {"\u2002", "&ensp;"}, // en space, U+2002 ISOpub -->
         {"\u2003", "&emsp;"}, // em space, U+2003 ISOpub -->
         {"\u2009", "&thinsp;"}, // thin space, U+2009 ISOpub -->
-        {"\u200C", "&zwnj;"}, // zero width non-joiner,U+200C NEW RFC 2070 -->
+        {"\u200C", "&zwnj;"}, // zero width non-joiner, U+200C NEW RFC 2070 -->
         {"\u200D", "&zwj;"}, // zero width joiner, U+200D NEW RFC 2070 -->
         {"\u200E", "&lrm;"}, // left-to-right mark, U+200E NEW RFC 2070 -->
         {"\u200F", "&rlm;"}, // right-to-left mark, U+200F NEW RFC 2070 -->
         {"\u2013", "&ndash;"}, // en dash, U+2013 ISOpub -->
         {"\u2014", "&mdash;"}, // em dash, U+2014 ISOpub -->
-        {"\u2018", "&lsquo;"}, // left single quotation mark,U+2018 ISOnum -->
-        {"\u2019", "&rsquo;"}, // right single quotation mark,U+2019 ISOnum -->
+        {"\u2018", "&lsquo;"}, // left single quotation mark, U+2018 ISOnum -->
+        {"\u2019", "&rsquo;"}, // right single quotation mark, U+2019 ISOnum -->
         {"\u201A", "&sbquo;"}, // single low-9 quotation mark, U+201A NEW -->
-        {"\u201C", "&ldquo;"}, // left double quotation mark,U+201C ISOnum -->
-        {"\u201D", "&rdquo;"}, // right double quotation mark,U+201D ISOnum -->
+        {"\u201C", "&ldquo;"}, // left double quotation mark, U+201C ISOnum -->
+        {"\u201D", "&rdquo;"}, // right double quotation mark, U+201D ISOnum -->
         {"\u201E", "&bdquo;"}, // double low-9 quotation mark, U+201E NEW -->
         {"\u2020", "&dagger;"}, // dagger, U+2020 ISOpub -->
         {"\u2021", "&Dagger;"}, // double dagger, U+2021 ISOpub -->
         {"\u2030", "&permil;"}, // per mille sign, U+2030 ISOtech -->
-        {"\u2039", "&lsaquo;"}, // single left-pointing angle quotation mark,U+2039 ISO proposed -->
+        {"\u2039", "&lsaquo;"}, // single left-pointing angle quotation mark, U+2039 ISO proposed -->
         // <!-- lsaquo is proposed but not yet ISO standardized -->
-        {"\u203A", "&rsaquo;"}, // single right-pointing angle quotation mark,U+203A ISO proposed -->
+        {"\u203A", "&rsaquo;"}, // single right-pointing angle quotation mark, U+203A ISO proposed -->
         // <!-- rsaquo is proposed but not yet ISO standardized -->
         {"\u20AC", "&euro;"}, // -- euro sign, U+20AC NEW -->
     };
@@ -448,7 +448,7 @@ public class EntityArrays {
      */
     public static String[][] invert(final String[][] array) {
         final String[][] newarray = new String[array.length][2];
-        for(int i = 0; i<array.length; i++) {
+        for (int i = 0; i<array.length; i++) {
             newarray[i][0] = array[i][1];
             newarray[i][1] = array[i][0];
         }

http://git-wip-us.apache.org/repos/asf/commons-lang/blob/89cbfa54/src/main/java/org/apache/commons/lang3/text/translate/NumericEntityEscaper.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/commons/lang3/text/translate/NumericEntityEscaper.java b/src/main/java/org/apache/commons/lang3/text/translate/NumericEntityEscaper.java
index 83f6604..dcb2c3a 100644
--- a/src/main/java/org/apache/commons/lang3/text/translate/NumericEntityEscaper.java
+++ b/src/main/java/org/apache/commons/lang3/text/translate/NumericEntityEscaper.java
@@ -104,7 +104,7 @@ public class NumericEntityEscaper extends CodePointTranslator {
      */
     @Override
     public boolean translate(final int codepoint, final Writer out) throws IOException {
-        if(between) {
+        if (between) {
             if (codepoint < below || codepoint > above) {
                 return false;
             }

http://git-wip-us.apache.org/repos/asf/commons-lang/blob/89cbfa54/src/main/java/org/apache/commons/lang3/text/translate/NumericEntityUnescaper.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/commons/lang3/text/translate/NumericEntityUnescaper.java b/src/main/java/org/apache/commons/lang3/text/translate/NumericEntityUnescaper.java
index 91e2e98..06994fe 100644
--- a/src/main/java/org/apache/commons/lang3/text/translate/NumericEntityUnescaper.java
+++ b/src/main/java/org/apache/commons/lang3/text/translate/NumericEntityUnescaper.java
@@ -57,7 +57,7 @@ public class NumericEntityUnescaper extends CharSequenceTranslator {
      * @param options to apply to this unescaper
      */
     public NumericEntityUnescaper(final OPTION... options) {
-        if(options.length > 0) {
+        if (options.length > 0) {
             this.options = EnumSet.copyOf(Arrays.asList(options));
         } else {
             this.options = EnumSet.copyOf(Arrays.asList(new OPTION[] { OPTION.semiColonRequired }));
@@ -81,24 +81,24 @@ public class NumericEntityUnescaper extends CharSequenceTranslator {
     public int translate(final CharSequence input, final int index, final Writer out) throws IOException {
         final int seqEnd = input.length();
         // Uses -2 to ensure there is something after the &#
-        if(input.charAt(index) == '&' && index < seqEnd - 2 && input.charAt(index + 1) == '#') {
+        if (input.charAt(index) == '&' && index < seqEnd - 2 && input.charAt(index + 1) == '#') {
             int start = index + 2;
             boolean isHex = false;
 
             final char firstChar = input.charAt(start);
-            if(firstChar == 'x' || firstChar == 'X') {
+            if (firstChar == 'x' || firstChar == 'X') {
                 start++;
                 isHex = true;
 
                 // Check there's more than just an x after the &#
-                if(start == seqEnd) {
+                if (start == seqEnd) {
                     return 0;
                 }
             }
 
             int end = start;
             // Note that this supports character codes without a ; on the end
-            while(end < seqEnd && ( input.charAt(end) >= '0' && input.charAt(end) <= '9' ||
+            while (end < seqEnd && ( input.charAt(end) >= '0' && input.charAt(end) <= '9' ||
                                     input.charAt(end) >= 'a' && input.charAt(end) <= 'f' ||
                                     input.charAt(end) >= 'A' && input.charAt(end) <= 'F' ) ) {
                 end++;
@@ -106,18 +106,18 @@ public class NumericEntityUnescaper extends CharSequenceTranslator {
 
             final boolean semiNext = end != seqEnd && input.charAt(end) == ';';
 
-            if(!semiNext) {
-                if(isSet(OPTION.semiColonRequired)) {
+            if (!semiNext) {
+                if (isSet(OPTION.semiColonRequired)) {
                     return 0;
                 } else
-                if(isSet(OPTION.errorIfNoSemiColon)) {
+                if (isSet(OPTION.errorIfNoSemiColon)) {
                     throw new IllegalArgumentException("Semi-colon required at end of numeric entity");
                 }
             }
 
             int entityValue;
             try {
-                if(isHex) {
+                if (isHex) {
                     entityValue = Integer.parseInt(input.subSequence(start, end).toString(), 16);
                 } else {
                     entityValue = Integer.parseInt(input.subSequence(start, end).toString(), 10);
@@ -126,7 +126,7 @@ public class NumericEntityUnescaper extends CharSequenceTranslator {
                 return 0;
             }
 
-            if(entityValue > 0xFFFF) {
+            if (entityValue > 0xFFFF) {
                 final char[] chars = Character.toChars(entityValue);
                 out.write(chars[0]);
                 out.write(chars[1]);