Issue checkstyle#3048: ParenPad now correctly evaluates precedence pa…

…rens
subkrish · Mar 21, 2017 · 7718c0a · 7718c0a
1 parent e763212
commit 7718c0a
Showing 1 changed file with 13 additions and 3 deletions.
diff --git a/src/main/java/com/puppycrawl/tools/checkstyle/checks/whitespace/ParenPadCheck.java b/src/main/java/com/puppycrawl/tools/checkstyle/checks/whitespace/ParenPadCheck.java
@@ -181,11 +181,21 @@ private void processExpression(DetailAST ast) {
             DetailAST childAst = ast.getFirstChild();
             while (childAst != null) {
                 if (childAst.getType() == TokenTypes.LPAREN) {
-                    processLeft(childAst);
-                    processExpression(childAst);
+                    if (childAst.getNextSibling().getType() == TokenTypes.TYPECAST) {
+                        processExpression(childAst);
+                    }
+                    else {
+                        processLeft(childAst);
+                        processExpression(childAst);
+                    }
                 }
                 else if (childAst.getType() == TokenTypes.RPAREN && !isInTypecast(childAst)) {
-                    processRight(childAst);
+                    if (childAst.getPreviousSibling().getType() == TokenTypes.TYPECAST) {
+                        processExpression(childAst);
+                    }
+                    else {
+                        processRight(childAst);
+                    }
                 }
                 else if (!isAcceptableToken(childAst)) {
                     //Traverse all subtree tokens which will never be configured