Description Benno Baumgartner 2006-02-17 10:30:19 EST

Version: 3.2.0
Build id: I20060216-2000

Given:
if (i != 0 //I don't like 0
		&& i != 10)
	System.out.println(i);

1. Select:
(i != 0 //I don't like 0
		&& i != 10)
2. Ctrl-1
3. Add paranoiac paranthesis
Result:
if ((i != 0 //I don't like 0)
		&& (i != 10))
	System.out.println(i);

Expected:
if ((i != 0) //I don't like 0
		&& (i != 10))
	System.out.println(i);

(the ')' is inserted at end of comment)

Looks like a bug in AST rewrite. The code to add parenthesis is rather simple:

ParenthesizedExpression parenthesizedExpression=
      ast.newParenthesizedExpression();
parenthesizedExpression.setExpression(
      (Expression) rewrite.createCopyTarget(expression));
rewrite.replace(expression, parenthesizedExpression, group);