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

4 files changed, 26 insertions, 26 deletions

diff --git a/src/test/java/org/apache/commons/lang3/builder/CompareToBuilderTest.java b/src/test/java/org/apache/commons/lang3/builder/CompareToBuilderTest.java
index ef3b7c534..950b66d8d 100644
--- a/src/test/java/org/apache/commons/lang3/builder/CompareToBuilderTest.java
+++ b/src/test/java/org/apache/commons/lang3/builder/CompareToBuilderTest.java
@@ -218,7 +218,7 @@ public class CompareToBuilderTest {
 
         // transitive
         if (CompareToBuilder.reflectionCompare(x, y, testTransients, null, excludeFields) > 0
-                && CompareToBuilder.reflectionCompare(y, z, testTransients, null, excludeFields) > 0){
+                && CompareToBuilder.reflectionCompare(y, z, testTransients, null, excludeFields) > 0) {
             assertTrue(CompareToBuilder.reflectionCompare(x, z, testTransients, null, excludeFields) > 0);
         }
 
@@ -745,9 +745,9 @@ public class CompareToBuilderTest {
         final short[][] array3 = new short[2][3];
         for (short i = 0; i < array1.length; ++i) {
             for (short j = 0; j < array1[0].length; j++) {
-                array1[i][j] = (short)((i + 1) * (j + 1));
-                array2[i][j] = (short)((i + 1) * (j + 1));
-                array3[i][j] = (short)((i + 1) * (j + 1));
+                array1[i][j] = (short) ((i + 1) * (j + 1));
+                array2[i][j] = (short) ((i + 1) * (j + 1));
+                array3[i][j] = (short) ((i + 1) * (j + 1));
             }
         }
         array3[1][2] = 100;
@@ -769,9 +769,9 @@ public class CompareToBuilderTest {
         final char[][] array3 = new char[2][3];
         for (short i = 0; i < array1.length; ++i) {
             for (short j = 0; j < array1[0].length; j++) {
-                array1[i][j] = (char)((i + 1) * (j + 1));
-                array2[i][j] = (char)((i + 1) * (j + 1));
-                array3[i][j] = (char)((i + 1) * (j + 1));
+                array1[i][j] = (char) ((i + 1) * (j + 1));
+                array2[i][j] = (char) ((i + 1) * (j + 1));
+                array3[i][j] = (char) ((i + 1) * (j + 1));
             }
         }
         array3[1][2] = 100;
@@ -793,9 +793,9 @@ public class CompareToBuilderTest {
         final byte[][] array3 = new byte[2][3];
         for (byte i = 0; i < array1.length; ++i) {
             for (byte j = 0; j < array1[0].length; j++) {
-                array1[i][j] = (byte)((i + 1) * (j + 1));
-                array2[i][j] = (byte)((i + 1) * (j + 1));
-                array3[i][j] = (byte)((i + 1) * (j + 1));
+                array1[i][j] = (byte) ((i + 1) * (j + 1));
+                array2[i][j] = (byte) ((i + 1) * (j + 1));
+                array3[i][j] = (byte) ((i + 1) * (j + 1));
             }
         }
         array3[1][2] = 100;
diff --git a/src/test/java/org/apache/commons/lang3/builder/JsonToStringStyleTest.java b/src/test/java/org/apache/commons/lang3/builder/JsonToStringStyleTest.java
index 1e22842e3..3cf8f5887 100644
--- a/src/test/java/org/apache/commons/lang3/builder/JsonToStringStyleTest.java
+++ b/src/test/java/org/apache/commons/lang3/builder/JsonToStringStyleTest.java
@@ -197,9 +197,9 @@ public class JsonToStringStyleTest {
 
     @Test
     public void testNestingPerson() {
-        final Person p = new Person(){
+        final Person p = new Person() {
             @Override
-            public String toString(){
+            public String toString() {
                 return new ToStringBuilder(this).append("name", this.name)
                     .append("age", this.age).append("smoker", this.smoker)
                     .toString();
@@ -301,16 +301,16 @@ public class JsonToStringStyleTest {
 
     @Test
     public void testLANG1395() {
-        assertEquals("{\"name\":\"value\"}",new ToStringBuilder(base).append("name","value").toString());
-        assertEquals("{\"name\":\"\"}",new ToStringBuilder(base).append("name","").toString());
-        assertEquals("{\"name\":\"\\\"\"}",new ToStringBuilder(base).append("name",'"').toString());
-        assertEquals("{\"name\":\"\\\\\"}",new ToStringBuilder(base).append("name",'\\').toString());
-        assertEquals("{\"name\":\"Let's \\\"quote\\\" this\"}",new ToStringBuilder(base).append("name","Let's \"quote\" this").toString());
+        assertEquals("{\"name\":\"value\"}", new ToStringBuilder(base).append("name", "value").toString());
+        assertEquals("{\"name\":\"\"}", new ToStringBuilder(base).append("name", "").toString());
+        assertEquals("{\"name\":\"\\\"\"}", new ToStringBuilder(base).append("name", '"').toString());
+        assertEquals("{\"name\":\"\\\\\"}", new ToStringBuilder(base).append("name", '\\').toString());
+        assertEquals("{\"name\":\"Let's \\\"quote\\\" this\"}", new ToStringBuilder(base).append("name", "Let's \"quote\" this").toString());
     }
 
     @Test
     public void testLANG1396() {
-        assertEquals("{\"Let's \\\"quote\\\" this\":\"value\"}",new ToStringBuilder(base).append("Let's \"quote\" this","value").toString());
+        assertEquals("{\"Let's \\\"quote\\\" this\":\"value\"}", new ToStringBuilder(base).append("Let's \"quote\" this", "value").toString());
     }
 
     /**
diff --git a/src/test/java/org/apache/commons/lang3/builder/ReflectionToStringBuilderExcludeNullValuesTest.java b/src/test/java/org/apache/commons/lang3/builder/ReflectionToStringBuilderExcludeNullValuesTest.java
index 8ddf9b4b7..c97d7e5f7 100644
--- a/src/test/java/org/apache/commons/lang3/builder/ReflectionToStringBuilderExcludeNullValuesTest.java
+++ b/src/test/java/org/apache/commons/lang3/builder/ReflectionToStringBuilderExcludeNullValuesTest.java
@@ -44,7 +44,7 @@ public class ReflectionToStringBuilderExcludeNullValuesTest {
     private final TestFixture BOTH_NULL = new TestFixture(null, null);
 
     @Test
-    public void test_NonExclude(){
+    public void test_NonExclude() {
         //normal case=
         String toString = ReflectionToStringBuilder.toString(BOTH_NON_NULL, null, false, false, false, null);
         assertTrue(toString.contains(INTEGER_FIELD_NAME));
@@ -67,7 +67,7 @@ public class ReflectionToStringBuilderExcludeNullValuesTest {
     }
 
     @Test
-    public void test_excludeNull(){
+    public void test_excludeNull() {
 
         //test normal case
         String toString = ReflectionToStringBuilder.toString(BOTH_NON_NULL, null, false, false, true, null);
@@ -91,7 +91,7 @@ public class ReflectionToStringBuilderExcludeNullValuesTest {
     }
 
     @Test
-    public void test_ConstructorOption(){
+    public void test_ConstructorOption() {
         ReflectionToStringBuilder builder = new ReflectionToStringBuilder(BOTH_NON_NULL, null, null, null, false, false, true);
         assertTrue(builder.isExcludeNullValues());
         String toString = builder.toString();
@@ -115,7 +115,7 @@ public class ReflectionToStringBuilderExcludeNullValuesTest {
     }
 
     @Test
-    public void test_ConstructorOptionNormal(){
+    public void test_ConstructorOptionNormal() {
         final ReflectionToStringBuilder builder = new ReflectionToStringBuilder(BOTH_NULL, null, null, null, false, false, false);
         assertFalse(builder.isExcludeNullValues());
         String toString = builder.toString();
@@ -140,7 +140,7 @@ public class ReflectionToStringBuilderExcludeNullValuesTest {
     }
 
     @Test
-    public void test_ConstructorOption_ExcludeNull(){
+    public void test_ConstructorOption_ExcludeNull() {
         ReflectionToStringBuilder builder = new ReflectionToStringBuilder(BOTH_NULL, null, null, null, false, false, false);
         builder.setExcludeNullValues(true);
         assertTrue(builder.isExcludeNullValues());
diff --git a/src/test/java/org/apache/commons/lang3/builder/ToStringBuilderTest.java b/src/test/java/org/apache/commons/lang3/builder/ToStringBuilderTest.java
index 3366721e7..da973e7ca 100644
--- a/src/test/java/org/apache/commons/lang3/builder/ToStringBuilderTest.java
+++ b/src/test/java/org/apache/commons/lang3/builder/ToStringBuilderTest.java
@@ -46,7 +46,7 @@ public class ToStringBuilderTest {
      * All tests should leave the registry empty.
      */
     @AfterEach
-    public void after(){
+    public void after() {
         validateNullToStringStyleRegistry();
     }
 
@@ -512,7 +512,7 @@ public class ToStringBuilderTest {
             this.typeIsSelf = this;
         }
 
-        public String getOtherType(){
+        public String getOtherType() {
             return this.otherType;
         }
 
@@ -845,7 +845,7 @@ public class ToStringBuilderTest {
     }
 
     @Test
-    public void testConstructToStringBuilder(){
+    public void testConstructToStringBuilder() {
         final ToStringBuilder stringBuilder1 = new ToStringBuilder(base, null, null);
         final ToStringBuilder stringBuilder2 = new ToStringBuilder(base, ToStringStyle.DEFAULT_STYLE, new StringBuffer(1024));
         assertEquals(ToStringStyle.DEFAULT_STYLE, stringBuilder1.getStyle());