Revert "Checkstyle: no space inside parentheses"

This reverts commit 28204b1caf363e7f7ae8d184ad305cf958a24608.

Breaks controller, we certainly do not need any more breakage at this point.

Change-Id: I3ee127e638a1adcbdf295ee7ab3e7328c6fd5777
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

diff --git a/bundles-test/src/main/java/org/opendaylight/odlparent/bundlestest/BundleDiagInfos.java b/bundles-test/src/main/java/org/opendaylight/odlparent/bundlestest/BundleDiagInfos.java
index 8a53d68b88edafd68ed4fe8d433635b3769fc31e..547e14999e1dafc71f15342a7e0c4ca94946ca68 100644 (file)
--- a/bundles-test/src/main/java/org/opendaylight/odlparent/bundlestest/BundleDiagInfos.java
+++ b/bundles-test/src/main/java/org/opendaylight/odlparent/bundlestest/BundleDiagInfos.java
@@ -36,7 +36,7 @@ public class BundleDiagInfos {
     private final Map<BundleState, Integer> bundleStatesCounters;
 
     private static final Map<String, BundleState> WHITELISTED_BUNDLES = ImmutableMap.of(
-            "slf4j.log4j12", Installed);
+            "slf4j.log4j12", Installed );
 
     public static BundleDiagInfos forContext(BundleContext bundleContext, BundleService bundleService) {
         List<String> okBundleStateInfoTexts = new ArrayList<>();

diff --git a/bundles-test/src/main/java/org/opendaylight/odlparent/bundlestest/TestBundleDiag.java b/bundles-test/src/main/java/org/opendaylight/odlparent/bundlestest/TestBundleDiag.java
index 7a0732ff4f00f72e9120131e49b6a197a192a474..4d72552febc8c65a2b2ec5fd2be496fab7008f78 100644 (file)
--- a/bundles-test/src/main/java/org/opendaylight/odlparent/bundlestest/TestBundleDiag.java
+++ b/bundles-test/src/main/java/org/opendaylight/odlparent/bundlestest/TestBundleDiag.java
@@ -57,7 +57,7 @@ public class TestBundleDiag {
                     .conditionEvaluationListener(
                         condition -> LOG.info("{} (elapsed time {}s, remaining time {}s)",
                             ((BundleDiagInfos) condition.getValue()).getSummaryText(),
-                            condition.getElapsedTimeInMS() / 1000, condition.getRemainingTimeInMS() / 1000))
+                            condition.getElapsedTimeInMS() / 1000, condition.getRemainingTimeInMS() / 1000 ))
                     .until(() -> getBundleDiagInfos(), new BundleServiceSummaryMatcher());
 
             // If we're here then either BundleServiceSummaryMatcher quit because of Active, Failure or Stopping..

diff --git a/bundles4-test/src/main/java/org/opendaylight/odlparent/bundles4test/BundleDiagInfos.java b/bundles4-test/src/main/java/org/opendaylight/odlparent/bundles4test/BundleDiagInfos.java
index 47d698c5a4d04190a12df6b0856d394216cfeed2..47f6f6e536d28d8c17fc4d9f2ef5b4d21ba3a010 100644 (file)
--- a/bundles4-test/src/main/java/org/opendaylight/odlparent/bundles4test/BundleDiagInfos.java
+++ b/bundles4-test/src/main/java/org/opendaylight/odlparent/bundles4test/BundleDiagInfos.java
@@ -36,7 +36,7 @@ public class BundleDiagInfos {
     private final Map<BundleState, Integer> bundleStatesCounters;
 
     private static final Map<String, BundleState> WHITELISTED_BUNDLES = ImmutableMap.of(
-            "slf4j.log4j12", Installed);
+            "slf4j.log4j12", Installed );
 
     public static BundleDiagInfos forContext(BundleContext bundleContext, BundleService bundleService) {
         List<String> okBundleStateInfoTexts = new ArrayList<>();

diff --git a/bundles4-test/src/main/java/org/opendaylight/odlparent/bundles4test/TestBundleDiag.java b/bundles4-test/src/main/java/org/opendaylight/odlparent/bundles4test/TestBundleDiag.java
index 88a585b3762de2aed596e6f63040b6b037f4ab63..d1b87d91b23331bb0dd5054fa9ef481d4b18d9e5 100644 (file)
--- a/bundles4-test/src/main/java/org/opendaylight/odlparent/bundles4test/TestBundleDiag.java
+++ b/bundles4-test/src/main/java/org/opendaylight/odlparent/bundles4test/TestBundleDiag.java
@@ -57,7 +57,7 @@ public class TestBundleDiag {
                     .conditionEvaluationListener(
                         condition -> LOG.info("{} (elapsed time {}s, remaining time {}s)",
                             ((BundleDiagInfos) condition.getValue()).getSummaryText(),
-                            condition.getElapsedTimeInMS() / 1000, condition.getRemainingTimeInMS() / 1000))
+                            condition.getElapsedTimeInMS() / 1000, condition.getRemainingTimeInMS() / 1000 ))
                     .until(() -> getBundleDiagInfos(), new BundleServiceSummaryMatcher());
 
             // If we're here then either BundleServiceSummaryMatcher quit because of Active, Failure or Stopping..

diff --git a/checkstyle/src/main/resources/odl_checks.xml b/checkstyle/src/main/resources/odl_checks.xml
index f1c23365b16285a2f0bb00bb3fc77516c0ae9277..db4f698d0cf1b467145a702f2ac9c86f9b43ddf0 100644 (file)
--- a/checkstyle/src/main/resources/odl_checks.xml
+++ b/checkstyle/src/main/resources/odl_checks.xml
@@ -75,7 +75,6 @@
              <message key="ws.notPreceded"
              value="WhitespaceAround: ''{0}'' is not preceded with whitespace."/>
         </module>
-        <module name="ParenPad" />
         <module name="OneStatementPerLine"/>
         <module name="MultipleVariableDeclarations"/>
         <module name="ArrayTypeStyle"/>