IDEA-52305 condensed code formatting for "flying geese" braces
authorDenis Zhdanov <Denis.Zhdanov@jetbrains.com>
Tue, 27 Apr 2010 08:35:14 +0000 (12:35 +0400)
committerDenis Zhdanov <Denis.Zhdanov@jetbrains.com>
Tue, 27 Apr 2010 08:35:14 +0000 (12:35 +0400)
1. Adopted utility method name change;
2. Suppression for broken 'constant condition' expression is added;

java/java-impl/src/com/intellij/psi/formatter/java/AbstractJavaBlock.java

index d81d710a97cece278e4ed9896ee71fe5e41586d6..65819311954b8cead6bc822b664fc2fbdb131a71 100644 (file)
@@ -166,7 +166,7 @@ public abstract class AbstractJavaBlock extends AbstractBlock implements JavaBlo
       }
     }
 
-    final ASTNode prevElement = FormattingAstUtil.getPrevElement(child);
+    final ASTNode prevElement = FormattingAstUtil.getPrevNonWhiteSpaceNode(child);
     if (prevElement != null && prevElement.getElementType() == JavaElementType.MODIFIER_LIST) {
       return Indent.getNoneIndent();
     }
@@ -561,6 +561,7 @@ public abstract class AbstractJavaBlock extends AbstractBlock implements JavaBlo
     return role == ChildRole.OPERATION_SIGN || role == ChildRole.COLON;
   }
 
+  @SuppressWarnings({"ConstantConditions"})
   private Block createMethodCallExpressionBlock(final ASTNode node, final Wrap blockWrap, final Alignment alignment) {
     final ArrayList<ASTNode> nodes = new ArrayList<ASTNode>();
     final ArrayList<Block> subBlocks = new ArrayList<Block>();