Bug 74580 formatting for condition quick assists
3.1 M2 test pass The formatting of conditionals modified by the new quick assists is misleading. Consider the following: if (c1) { one(); } else if (c2) { two(); } else { three(); } translates into: if (!c1) if (c2) { two(); } else { three(); } else { one(); } which is correct, but it is barely visible from the formatting that the last else clause belongs to the first if statement. I would expect this (with or without the braces): if (!c1) if (c2) { two(); } else { three(); } else { one(); }
org.eclipse.jdt.ui/ui/org/eclipse/jdt/internal/ui/text/correction/AdvancedQuickAssistProcessor.java
ITypeBinding typeBinding = coveredExpression.resolveTypeBinding();
			if ((typeBinding == null) || !typeBinding.getName().equals("boolean")) { 
				continue;
			}
			
			Expression inversedExpression = getInversedBooleanExpression(ast, rewrite, coveredExpression);
			rewrite.replace(coveredExpression, inversedExpression, null);
			hasChanges= true;
		}
		
		if (!hasChanges) {
			return false;
		}
		if (resultingCollections == null) {
			return true;
		}
		
		
		
		String label = CorrectionMessages.getString("AdvancedQuickAssistProcessor.inverseConditions.description"); 
		Image image = JavaPluginImages.get(JavaPluginImages.IMG_CORRECTION_CHANGE);
		ASTRewriteCorrectionProposal proposal = new ASTRewriteCorrectionProposal(label, context.getCompilationUnit(),
				rewrite, 1, image);
		resultingCollections.add(proposal);
		return true;
	}
	private static Expression getInversedBooleanExpression(AST ast, ASTRewrite rewrite, Expression expression) {
		if (expression instanceof InfixExpression) {
			InfixExpression infixExpression = (InfixExpression) expression;
			InfixExpression.Operator operator = infixExpression.getOperator();