Attachment 189391 Details for Bug 335173 – final fix + tests

Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

[patch] final fix + tests

correct number of parentheses65.txt (text/plain), 171.99 KB, created by Deepak Azad

on 2011-02-20 23:23:48 EST

(hide)

Description:

Filename:

MIME Type:

Creator: Deepak Azad

Created: 2011-02-20 23:23:48 EST

Size: 171.99 KB

patch

obsolete

>### Eclipse Workspace Patch 1.0
>#P org.eclipse.jdt.ui
>Index: core extension/org/eclipse/jdt/internal/corext/codemanipulation/GetterSetterUtil.java
>===================================================================
>RCS file: /cvsroot/eclipse/org.eclipse.jdt.ui/core extension/org/eclipse/jdt/internal/corext/codemanipulation/GetterSetterUtil.java,v
>retrieving revision 1.29
>diff -u -r1.29 GetterSetterUtil.java
>--- core extension/org/eclipse/jdt/internal/corext/codemanipulation/GetterSetterUtil.java	16 Feb 2009 11:52:17 -0000	1.29
>+++ core extension/org/eclipse/jdt/internal/corext/codemanipulation/GetterSetterUtil.java	21 Feb 2011 04:10:01 -0000
>@@ -1,5 +1,5 @@
> /*******************************************************************************
>- * Copyright (c) 2000, 2009 IBM Corporation and others.
>+ * Copyright (c) 2000, 2011 IBM Corporation and others.
> * All rights reserved. This program and the accompanying materials
> * are made available under the terms of the Eclipse Public License v1.0
> * which accompanies this distribution, and is available at
>@@ -23,6 +23,7 @@
> import org.eclipse.jdt.core.dom.AST;
> import org.eclipse.jdt.core.dom.ASTNode;
> import org.eclipse.jdt.core.dom.Assignment;
>+import org.eclipse.jdt.core.dom.Assignment.Operator;
> import org.eclipse.jdt.core.dom.CastExpression;
> import org.eclipse.jdt.core.dom.Expression;
> import org.eclipse.jdt.core.dom.ITypeBinding;
>@@ -33,10 +34,10 @@
> import org.eclipse.jdt.core.dom.PostfixExpression;
> import org.eclipse.jdt.core.dom.PrefixExpression;
> import org.eclipse.jdt.core.dom.PrimitiveType;
>-import org.eclipse.jdt.core.dom.Assignment.Operator;
> import org.eclipse.jdt.core.dom.rewrite.ASTRewrite;
> 
> import org.eclipse.jdt.internal.corext.dom.ASTNodes;
>+import org.eclipse.jdt.internal.corext.dom.NecessaryParenthesesChecker;
> import org.eclipse.jdt.internal.corext.util.JavaModelUtil;
> import org.eclipse.jdt.internal.corext.util.JdtFlags;
> 
>@@ -260,8 +261,14 @@
> 				InfixExpression infix= ast.newInfixExpression();
> 				infix.setLeftOperand(getterExpression);
> 				infix.setOperator(ASTNodes.convertToInfixOperator(assignment.getOperator()));
>-				infix.setRightOperand(copiedRightOp);
> 				ITypeBinding infixType= infix.resolveTypeBinding();
>+				if (NecessaryParenthesesChecker.needsParentheses(copiedRightOp, infix, InfixExpression.RIGHT_OPERAND_PROPERTY)) {
>+					//TODO: this introduces extra parentheses as the new "infix" node doesn't have bindings
>+					ParenthesizedExpression p= ast.newParenthesizedExpression();
>+					p.setExpression(copiedRightOp);
>+					copiedRightOp= p;
>+				}
>+				infix.setRightOperand(copiedRightOp);
> 				return createNarrowCastIfNessecary(infix, infixType, ast, variableType, is50OrHigher);
> 			}
> 		} else if (node.getNodeType() == ASTNode.POSTFIX_EXPRESSION) {
>@@ -336,7 +343,7 @@
> 			castTo= ast.newPrimitiveType(PrimitiveType.SHORT);
> 		if (castTo != null) {
> 			CastExpression cast= ast.newCastExpression();
>-			if (ASTNodes.needsParentheses(expression)) {
>+			if (NecessaryParenthesesChecker.needsParentheses(expression, cast, CastExpression.EXPRESSION_PROPERTY)) {
> 				ParenthesizedExpression parenthesized= ast.newParenthesizedExpression();
> 				parenthesized.setExpression(expression);
> 				cast.setExpression(parenthesized);
>Index: core extension/org/eclipse/jdt/internal/corext/dom/ASTNodes.java
>===================================================================
>RCS file: /cvsroot/eclipse/org.eclipse.jdt.ui/core extension/org/eclipse/jdt/internal/corext/dom/ASTNodes.java,v
>retrieving revision 1.106
>diff -u -r1.106 ASTNodes.java
>--- core extension/org/eclipse/jdt/internal/corext/dom/ASTNodes.java	5 Aug 2010 08:09:53 -0000	1.106
>+++ core extension/org/eclipse/jdt/internal/corext/dom/ASTNodes.java	21 Feb 2011 04:10:01 -0000
>@@ -1,5 +1,5 @@
> /*******************************************************************************
>- * Copyright (c) 2000, 2010 IBM Corporation and others.
>+ * Copyright (c) 2000, 2011 IBM Corporation and others.
> * All rights reserved. This program and the accompanying materials
> * are made available under the terms of the Eclipse Public License v1.0
> * which accompanies this distribution, and is available at
>@@ -42,7 +42,6 @@
> import org.eclipse.jdt.core.dom.ASTVisitor;
> import org.eclipse.jdt.core.dom.AbstractTypeDeclaration;
> import org.eclipse.jdt.core.dom.AnonymousClassDeclaration;
>-import org.eclipse.jdt.core.dom.ArrayAccess;
> import org.eclipse.jdt.core.dom.ArrayType;
> import org.eclipse.jdt.core.dom.Assignment;
> import org.eclipse.jdt.core.dom.BodyDeclaration;
>@@ -73,13 +72,10 @@
> import org.eclipse.jdt.core.dom.PrimitiveType;
> import org.eclipse.jdt.core.dom.QualifiedName;
> import org.eclipse.jdt.core.dom.QualifiedType;
>-import org.eclipse.jdt.core.dom.ReturnStatement;
> import org.eclipse.jdt.core.dom.SimpleName;
> import org.eclipse.jdt.core.dom.SimpleType;
> import org.eclipse.jdt.core.dom.SingleVariableDeclaration;
> import org.eclipse.jdt.core.dom.StructuralPropertyDescriptor;
>-import org.eclipse.jdt.core.dom.SwitchCase;
>-import org.eclipse.jdt.core.dom.SwitchStatement;
> import org.eclipse.jdt.core.dom.Type;
> import org.eclipse.jdt.core.dom.VariableDeclaration;
> import org.eclipse.jdt.core.dom.VariableDeclarationExpression;
>@@ -88,7 +84,6 @@
> import org.eclipse.jdt.core.dom.WhileStatement;
> 
> import org.eclipse.jdt.internal.corext.codemanipulation.StubUtility;
>-import org.eclipse.jdt.internal.corext.refactoring.code.OperatorPrecedence;
> import org.eclipse.jdt.internal.corext.util.CodeFormatterUtil;
> import org.eclipse.jdt.internal.corext.util.Strings;
> 
>@@ -429,58 +424,6 @@
> 			|| locationInParent == DoStatement.BODY_PROPERTY;
> 	}
> 
>-	public static boolean needsParentheses(Expression expression) {
>-		int type= expression.getNodeType();
>-		return type == ASTNode.INFIX_EXPRESSION || type == ASTNode.CONDITIONAL_EXPRESSION ||
>-			type == ASTNode.PREFIX_EXPRESSION || type == ASTNode.POSTFIX_EXPRESSION ||
>-			type == ASTNode.CAST_EXPRESSION || type == ASTNode.INSTANCEOF_EXPRESSION;
>-	}
>-
>-	/**
>-	 * Checks whether <code>substitute</code> must be parenthesized when used to replace
>-	 * <code>location</code>.
>-	 * 
>-	 * @param substitute substitute expression
>-	 * @param location expression to be replaced
>-	 * @return <code>true</code> iff <code>substitute</code> must be parenthesized when used to
>-	 * replace <code>location</code>
>-	 */
>-	public static boolean substituteMustBeParenthesized(Expression substitute, Expression location) {
>-		if (substitute instanceof Assignment) //for esthetic reasons
>-			return true;
>-
>- 	if (!needsParentheses(substitute))
>- 		return false;
>-
>- 	ASTNode parent= location.getParent();
>- 	StructuralPropertyDescriptor locationInParent= location.getLocationInParent();
>-		if (locationInParent instanceof ChildListPropertyDescriptor && locationInParent != InfixExpression.EXTENDED_OPERANDS_PROPERTY) {
>-			// e.g. argument lists of MethodInvocation, ClassInstanceCreation, ...
>- 			return false;
>- 	} else if (locationInParent == VariableDeclarationFragment.INITIALIZER_PROPERTY) {
>- 			return false;
>- 	} else if (locationInParent == SingleVariableDeclaration.INITIALIZER_PROPERTY) {
>- 		return false;
>- 	} else if (locationInParent == ReturnStatement.EXPRESSION_PROPERTY) {
>- 		return false;
>- 	} else if (locationInParent == EnhancedForStatement.EXPRESSION_PROPERTY) {
>- 		return false;
>- 	} else if (locationInParent == SwitchStatement.EXPRESSION_PROPERTY) {
>- 		return false;
>- 	} else if (locationInParent == SwitchCase.EXPRESSION_PROPERTY) {
>- 		return false;
>- 	} else if (locationInParent == ArrayAccess.INDEX_PROPERTY) {
>- 		return false;
>- 	} else if (parent instanceof Expression) {
>-			int substitutePrecedence= OperatorPrecedence.getExpressionPrecedence(substitute);
>-			int locationPrecedence= OperatorPrecedence.getExpressionPrecedence((Expression)parent);
>-			if (substitutePrecedence > locationPrecedence)
>-				return false;
>-		}
>-
>- return true;
>-	}
>-
> 	/**
> 	 * Returns the type to which an inlined variable initializer should be cast, or
> 	 * <code>null</code> if no cast is necessary.
>Index: core extension/org/eclipse/jdt/internal/corext/dom/NecessaryParenthesesChecker.java
>===================================================================
>RCS file: core extension/org/eclipse/jdt/internal/corext/dom/NecessaryParenthesesChecker.java
>diff -N core extension/org/eclipse/jdt/internal/corext/dom/NecessaryParenthesesChecker.java
>--- /dev/null	1 Jan 1970 00:00:00 -0000
>+++ core extension/org/eclipse/jdt/internal/corext/dom/NecessaryParenthesesChecker.java	1 Jan 1970 00:00:00 -0000
>@@ -0,0 +1,302 @@
>+/*******************************************************************************
>+ * Copyright (c) 2011 IBM Corporation and others.
>+ * All rights reserved. This program and the accompanying materials
>+ * are made available under the terms of the Eclipse Public License v1.0
>+ * which accompanies this distribution, and is available at
>+ * http://www.eclipse.org/legal/epl-v10.html
>+ *
>+ * Contributors:
>+ * IBM Corporation - initial API and implementation
>+ *******************************************************************************/
>+
>+package org.eclipse.jdt.internal.corext.dom;
>+
>+import java.util.Iterator;
>+
>+import org.eclipse.jdt.core.dom.ASTNode;
>+import org.eclipse.jdt.core.dom.ArrayAccess;
>+import org.eclipse.jdt.core.dom.AssertStatement;
>+import org.eclipse.jdt.core.dom.ChildListPropertyDescriptor;
>+import org.eclipse.jdt.core.dom.ConditionalExpression;
>+import org.eclipse.jdt.core.dom.DoStatement;
>+import org.eclipse.jdt.core.dom.EnhancedForStatement;
>+import org.eclipse.jdt.core.dom.Expression;
>+import org.eclipse.jdt.core.dom.ForStatement;
>+import org.eclipse.jdt.core.dom.ITypeBinding;
>+import org.eclipse.jdt.core.dom.IfStatement;
>+import org.eclipse.jdt.core.dom.InfixExpression;
>+import org.eclipse.jdt.core.dom.InfixExpression.Operator;
>+import org.eclipse.jdt.core.dom.ParenthesizedExpression;
>+import org.eclipse.jdt.core.dom.ReturnStatement;
>+import org.eclipse.jdt.core.dom.SingleVariableDeclaration;
>+import org.eclipse.jdt.core.dom.StructuralPropertyDescriptor;
>+import org.eclipse.jdt.core.dom.SwitchCase;
>+import org.eclipse.jdt.core.dom.SwitchStatement;
>+import org.eclipse.jdt.core.dom.SynchronizedStatement;
>+import org.eclipse.jdt.core.dom.ThrowStatement;
>+import org.eclipse.jdt.core.dom.VariableDeclarationFragment;
>+import org.eclipse.jdt.core.dom.WhileStatement;
>+
>+import org.eclipse.jdt.internal.corext.refactoring.code.OperatorPrecedence;
>+
>+/**
>+ * Helper class to check if an expression requires parentheses.
>+ * 
>+ * @since 3.7
>+ */
>+public class NecessaryParenthesesChecker {
>+
>+	/*
>+	 * Get the expression wrapped by the parentheses
>+	 * i.e. ((((expression)))) -> expression
>+	 */
>+	private static Expression getExpression(ParenthesizedExpression node) {
>+		Expression expression= node.getExpression();
>+		while (expression instanceof ParenthesizedExpression) {
>+			expression= ((ParenthesizedExpression)expression).getExpression();
>+		}
>+		return expression;
>+	}
>+
>+	private static boolean expressionTypeNeedsParentheses(Expression expression) {
>+		int type= expression.getNodeType();
>+		return type == ASTNode.INFIX_EXPRESSION
>+				|| type == ASTNode.CONDITIONAL_EXPRESSION
>+				|| type == ASTNode.PREFIX_EXPRESSION
>+				|| type == ASTNode.POSTFIX_EXPRESSION
>+				|| type == ASTNode.CAST_EXPRESSION
>+				|| type == ASTNode.INSTANCEOF_EXPRESSION
>+				|| type == ASTNode.ARRAY_CREATION
>+				|| type == ASTNode.ASSIGNMENT;
>+	}
>+
>+	private static boolean locationNeedsParentheses(StructuralPropertyDescriptor locationInParent) {
>+		if (locationInParent instanceof ChildListPropertyDescriptor && locationInParent != InfixExpression.EXTENDED_OPERANDS_PROPERTY) {
>+			// e.g. argument lists of MethodInvocation, ClassInstanceCreation, dimensions of ArrayCreation ...
>+			return false;
>+		}
>+		if (locationInParent == VariableDeclarationFragment.INITIALIZER_PROPERTY
>+				|| locationInParent == SingleVariableDeclaration.INITIALIZER_PROPERTY
>+				|| locationInParent == ReturnStatement.EXPRESSION_PROPERTY
>+				|| locationInParent == EnhancedForStatement.EXPRESSION_PROPERTY
>+				|| locationInParent == ForStatement.EXPRESSION_PROPERTY
>+				|| locationInParent == WhileStatement.EXPRESSION_PROPERTY
>+				|| locationInParent == DoStatement.EXPRESSION_PROPERTY
>+				|| locationInParent == AssertStatement.EXPRESSION_PROPERTY
>+				|| locationInParent == AssertStatement.MESSAGE_PROPERTY
>+				|| locationInParent == IfStatement.EXPRESSION_PROPERTY
>+				|| locationInParent == SwitchStatement.EXPRESSION_PROPERTY
>+				|| locationInParent == SwitchCase.EXPRESSION_PROPERTY
>+				|| locationInParent == ArrayAccess.INDEX_PROPERTY
>+				|| locationInParent == ThrowStatement.EXPRESSION_PROPERTY
>+				|| locationInParent == SynchronizedStatement.EXPRESSION_PROPERTY
>+				|| locationInParent == ParenthesizedExpression.EXPRESSION_PROPERTY) {
>+			return false;
>+		}
>+		return true;
>+	}
>+
>+	/*
>+	 * Do all operands in expression have same type
>+	 */
>+	private static boolean isAllOperandsHaveSameType(InfixExpression expression) {
>+		ITypeBinding binding= expression.getLeftOperand().resolveTypeBinding();
>+		if (binding == null)
>+			return false;
>+	
>+		ITypeBinding current= expression.getRightOperand().resolveTypeBinding();
>+		if (binding != current)
>+			return false;
>+	
>+		for (Iterator iterator= expression.extendedOperands().iterator(); iterator.hasNext();) {
>+			Expression operand= (Expression) iterator.next();
>+			current= operand.resolveTypeBinding();
>+			if (binding != current)
>+				return false;
>+		}
>+	
>+		return true;
>+	}
>+
>+	/*
>+	 * Is the expression of integer type
>+	 */
>+	private static boolean isExpressionIntegerType(Expression expression) {
>+		ITypeBinding binding= expression.resolveTypeBinding();
>+		if (binding == null)
>+			return false;
>+
>+		if (!binding.isPrimitive())
>+			return false;
>+
>+		String name= binding.getName();
>+		if (isIntegerNumber(name))
>+			return true;
>+
>+		return false;
>+	}
>+
>+	private static boolean isExpressionStringType(Expression expression) {
>+		ITypeBinding binding= expression.resolveTypeBinding();
>+		if (binding == null)
>+			return false;
>+
>+		return "java.lang.String".equals(binding.getQualifiedName()); //$NON-NLS-1$
>+	}
>+
>+	/*
>+	 * Is the given expression associative?
>+	 * 
>+	 * This is true if and only if: 
>+	 * <code>left operator (right) == (right) operator left == right operator left</code>
>+	 */
>+	private static boolean isAssociative(InfixExpression expression) {
>+		Operator operator= expression.getOperator();
>+
>+		if (operator == InfixExpression.Operator.PLUS)
>+			return isExpressionStringType(expression) || isExpressionIntegerType(expression) && isAllOperandsHaveSameType(expression);
>+
>+		if (operator == InfixExpression.Operator.TIMES)
>+			return isExpressionIntegerType(expression) && isAllOperandsHaveSameType(expression);
>+
>+		if (operator == InfixExpression.Operator.CONDITIONAL_AND
>+				|| operator == InfixExpression.Operator.CONDITIONAL_OR
>+				|| operator == InfixExpression.Operator.AND
>+				|| operator == InfixExpression.Operator.OR
>+				|| operator == InfixExpression.Operator.XOR)
>+			return true;
>+	
>+		return false;
>+	}
>+
>+	private static boolean isIntegerNumber(String name) {
>+		return "int".equals(name) || "long".equals(name) || "byte".equals(name) || "char".equals(name) || "short".equals(name); //$NON-NLS-1$ //$NON-NLS-2$ //$NON-NLS-3$ //$NON-NLS-4$ //$NON-NLS-5$
>+	}
>+
>+	private static boolean needsParenthesesInInfixExpression(Expression expression, InfixExpression parentInfix, StructuralPropertyDescriptor locationInParent) {
>+		if (locationInParent == InfixExpression.LEFT_OPERAND_PROPERTY) {
>+			//we have (expr op expr) op expr
>+			//infix expressions are evaluated from left to right -> parentheses not needed
>+			return false;
>+		} else if (isAssociative(parentInfix)) {
>+			//we have parent op (expr op expr) and op is associative
>+			//left op (right) == (right) op left == right op left
>+			if (expression instanceof InfixExpression) {
>+				InfixExpression infixExpression= (InfixExpression)expression;
>+				Operator operator= infixExpression.getOperator();
>+
>+				if (isExpressionStringType(parentInfix)) {
>+					if (parentInfix.getOperator() == InfixExpression.Operator.PLUS && operator == InfixExpression.Operator.PLUS && isExpressionStringType(infixExpression)) {
>+						// 1 + ("" + 2) == 1 + "" + 2
>+						// 1 + (2 + "") != 1 + 2 + ""
>+						// "" + (2 + "") == "" + 2 + ""
>+						return !isExpressionStringType(infixExpression.getLeftOperand()) && !isExpressionStringType(parentInfix.getLeftOperand());
>+					}
>+					//"" + (1 + 2), "" + (1 - 2) etc
>+					return true;
>+				}
>+
>+				if (parentInfix.getOperator() != InfixExpression.Operator.TIMES)
>+					return false;
>+	
>+				if (operator == InfixExpression.Operator.TIMES)
>+					// x * (y * z) == x * y * z
>+					return false;
>+	
>+				if (operator == InfixExpression.Operator.REMAINDER || operator == InfixExpression.Operator.DIVIDE)
>+					// x * (y % z) != x * y % z , x * (y / z) == x * y / z rounding involved
>+					return true;
>+	
>+				return false;
>+			}
>+			return false;
>+		} else {
>+			return true;
>+		}
>+	}
>+
>+	/**
>+	 * Can the parentheses be removed from the parenthesized expression ?
>+	 * 
>+	 * 
>+	 * Note: The parenthesized expression must not be an unparented node.
>+	 * 
>+	 * 
>+	 * @param expression the parenthesized expression
>+	 * @return <code>true</code> if parentheses can be removed, <code>false</code> otherwise.
>+	 */
>+	public static boolean canRemoveParentheses(Expression expression) {
>+		return canRemoveParentheses(expression, expression.getParent(), expression.getLocationInParent());
>+	}
>+
>+	/**
>+	 * Can the parentheses be removed from the parenthesized expression when inserted into
>+	 * <code>parent</code> at <code>locationInParent</code> ?
>+	 * 
>+	 * 
>+	 * Note: The parenthesized expression can be an unparented node.
>+	 * 
>+	 * 
>+	 * @param expression the parenthesized expression
>+	 * @param parent the parent node
>+	 * @param locationInParent location of expression in the parent
>+	 * @return <code>true</code> if parentheses can be removed, <code>false</code> otherwise.
>+	 */
>+	public static boolean canRemoveParentheses(Expression expression, ASTNode parent, StructuralPropertyDescriptor locationInParent) {
>+		if (!(expression instanceof ParenthesizedExpression)) {
>+			return false;
>+		}
>+		return !needsParentheses(getExpression((ParenthesizedExpression)expression), parent, locationInParent);
>+	}
>+
>+	/**
>+	 * Does the <code>expression</code> need parentheses when inserted into <code>parent</code> at
>+	 * <code>locationInParent</code> ?
>+	 * 
>+	 * 
>+	 * Note: The expression can be an unparented node.
>+	 * 
>+	 * 
>+	 * @param expression the expression
>+	 * @param parent the parent node
>+	 * @param locationInParent location of expression in the parent
>+	 * @return <code>true</code> if the expression needs parentheses, <code>false</code> otherwise.
>+	 */
>+	public static boolean needsParentheses(Expression expression, ASTNode parent, StructuralPropertyDescriptor locationInParent) {
>+		if (!expressionTypeNeedsParentheses(expression))
>+			return false;
>+
>+		if (!locationNeedsParentheses(locationInParent)) {
>+			return false;
>+		}
>+
>+		if (parent instanceof Expression) {
>+			Expression parentExpression= (Expression)parent;
>+
>+			int expressionPrecedence= OperatorPrecedence.getExpressionPrecedence(expression);
>+			int parentPrecedence= OperatorPrecedence.getExpressionPrecedence(parentExpression);
>+
>+			if (expressionPrecedence > parentPrecedence)
>+				//(opEx) opParent and opEx binds more -> parentheses not needed
>+				return false;
>+
>+			if (expressionPrecedence < parentPrecedence)
>+				//(opEx) opParent and opEx binds less -> parentheses needed
>+				return true;
>+
>+			//(opEx) opParent binds equal
>+
>+			if (parentExpression instanceof InfixExpression) {
>+				return needsParenthesesInInfixExpression(expression, (InfixExpression)parentExpression, locationInParent);
>+			}
>+
>+			if (parentExpression instanceof ConditionalExpression && locationInParent == ConditionalExpression.EXPRESSION_PROPERTY) {
>+				return true;
>+			}
>+
>+			return false;
>+		}
>+
>+		return true;
>+	}
>+}
>\ No newline at end of file
>Index: core extension/org/eclipse/jdt/internal/corext/fix/ExpressionsFix.java
>===================================================================
>RCS file: /cvsroot/eclipse/org.eclipse.jdt.ui/core extension/org/eclipse/jdt/internal/corext/fix/ExpressionsFix.java,v
>retrieving revision 1.18
>diff -u -r1.18 ExpressionsFix.java
>--- core extension/org/eclipse/jdt/internal/corext/fix/ExpressionsFix.java	29 Jan 2011 05:29:11 -0000	1.18
>+++ core extension/org/eclipse/jdt/internal/corext/fix/ExpressionsFix.java	21 Feb 2011 04:10:09 -0000
>@@ -12,7 +12,6 @@
> 
> import java.util.ArrayList;
> import java.util.HashSet;
>-import java.util.Iterator;
> 
> import org.eclipse.core.runtime.CoreException;
> 
>@@ -22,16 +21,13 @@
> import org.eclipse.jdt.core.dom.ASTNode;
> import org.eclipse.jdt.core.dom.ASTVisitor;
> import org.eclipse.jdt.core.dom.CompilationUnit;
>-import org.eclipse.jdt.core.dom.ConditionalExpression;
> import org.eclipse.jdt.core.dom.Expression;
>-import org.eclipse.jdt.core.dom.ITypeBinding;
> import org.eclipse.jdt.core.dom.InfixExpression;
>-import org.eclipse.jdt.core.dom.InfixExpression.Operator;
> import org.eclipse.jdt.core.dom.InstanceofExpression;
> import org.eclipse.jdt.core.dom.ParenthesizedExpression;
> import org.eclipse.jdt.core.dom.rewrite.ASTRewrite;
> 
>-import org.eclipse.jdt.internal.corext.refactoring.code.OperatorPrecedence;
>+import org.eclipse.jdt.internal.corext.dom.NecessaryParenthesesChecker;
> import org.eclipse.jdt.internal.corext.refactoring.structure.CompilationUnitRewrite;
> import org.eclipse.jdt.internal.corext.refactoring.util.NoCommentSourceRangeComputer;
> 
>@@ -86,169 +82,12 @@
> 		}
> 
> 		public boolean visit(ParenthesizedExpression node) {
>-			if (canRemoveParenthesis(node)) {
>+			if (NecessaryParenthesesChecker.canRemoveParentheses(node)) {
> 				fNodes.add(node);
> 			}
> 
> 			return true;
> 		}
>-
>-		/*
>-		 * Can the parenthesis around node be removed?
>-		 */
>-		private boolean canRemoveParenthesis(ParenthesizedExpression node) {
>-			ASTNode parent= node.getParent();
>-			if (!(parent instanceof Expression))
>-				return true;
>-
>-			Expression parentExpression= (Expression) parent;
>-			if (parentExpression instanceof ParenthesizedExpression)
>-				return true;
>-
>-			Expression expression= getExpression(node);
>-
>-			int expressionPrecedence= OperatorPrecedence.getExpressionPrecedence(expression);
>-			int parentPrecedence= OperatorPrecedence.getExpressionPrecedence(parentExpression);
>-
>-			if (expressionPrecedence > parentPrecedence)
>-				//(opEx) opParent and opEx binds more -> can safely remove
>-				return true;
>-
>-			if (expressionPrecedence < parentPrecedence)
>-				//(opEx) opParent and opEx binds less -> do not remove
>-				return false;
>-
>-			//(opEx) opParent binds equal
>-
>-			if (parentExpression instanceof InfixExpression) {
>-				InfixExpression parentInfix= (InfixExpression) parentExpression;
>-				if (parentInfix.getLeftOperand() == node) {
>-					//we have (expr op expr) op expr
>-					//infix expressions are evaluated from left to right -> can safely remove
>-					return true;
>-				} else if (isAssociative(parentInfix)) {
>-					//we have parent op (expr op expr) and op is associative
>-					//left op (right) == (right) op left == right op left
>-					if (expression instanceof InfixExpression) {
>-						InfixExpression infixExpression= (InfixExpression) expression;
>-						Operator operator= infixExpression.getOperator();
>-						if (parentInfix.getOperator() != InfixExpression.Operator.TIMES)
>-							return true;
>-
>-						if (operator == InfixExpression.Operator.TIMES)
>-							// x * (y * z) == x * y * z
>-							return true;
>-
>-						if (operator == InfixExpression.Operator.REMAINDER)
>-							// x * (y % z) != x * y % z
>-							return false;
>-
>-						//x * (y / z) == z * y / z iff no rounding
>-						ITypeBinding binding= infixExpression.resolveTypeBinding();
>-						if (binding == null)
>-							return false;
>-
>-						if (!binding.isPrimitive())
>-							return false;
>-
>-						String name= binding.getName();
>-						if (isIntegerNumber(name))
>-							//rounding involved
>-							return false;
>-
>-						return true;
>-					}
>-					return true;
>-				} else {
>-					return false;
>-				}
>-			} else if (parentExpression instanceof ConditionalExpression) {
>-				ConditionalExpression conditionalExpression= (ConditionalExpression) parentExpression;
>-				if (conditionalExpression.getElseExpression() != node)
>-					return false;
>-			}
>-
>-			return true;
>-		}
>-
>-		private boolean isIntegerNumber(String name) {
>-			return "int".equals(name) || "long".equals(name) || "byte".equals(name) || "char".equals(name) || "short".equals(name); //$NON-NLS-1$ //$NON-NLS-2$ //$NON-NLS-3$ //$NON-NLS-4$ //$NON-NLS-5$
>-		}
>-
>-		/*
>-		 * Get the expression wrapped by the parentheses
>-		 * i.e. ((((expression)))) -> expression
>-		 */
>-		private Expression getExpression(ParenthesizedExpression node) {
>-			Expression expression= node.getExpression();
>-			while (expression instanceof ParenthesizedExpression) {
>-				expression= ((ParenthesizedExpression) expression).getExpression();
>-			}
>-			return expression;
>-		}
>-
>-		/**
>-		 * Is the given expression associative?
>-		 * 
>-		 * This is true if and only if: 
>-		 * <code>left operator (right) == (right) operator left == right operator left</code>
>-		 * 
>-		 *
>-		 * @param expression the expression to inspect
>-		 * @return true if expression is associative
>-		 */
>-		public static boolean isAssociative(InfixExpression expression) {
>-			Operator operator= expression.getOperator();
>-			if (operator == InfixExpression.Operator.PLUS) {
>-				return isAllOperandsHaveSameType(expression);
>-			}
>-
>-			if (operator == Operator.LESS || operator == Operator.LESS_EQUALS || operator == Operator.GREATER || operator == Operator.GREATER_EQUALS) {
>-				return isAllOperandsHaveSameType(expression);
>-			}
>-
>-			if (operator == InfixExpression.Operator.CONDITIONAL_AND)
>-				return true;
>-
>-			if (operator == InfixExpression.Operator.CONDITIONAL_OR)
>-				return true;
>-
>-			if (operator == InfixExpression.Operator.AND)
>-				return true;
>-
>-			if (operator == InfixExpression.Operator.OR)
>-				return true;
>-
>-			if (operator == InfixExpression.Operator.XOR)
>-				return true;
>-
>-			if (operator == InfixExpression.Operator.TIMES)
>-				return true;
>-
>-			return false;
>-		}
>-
>-		/*
>-		 * Do all operands in expression have same type
>-		 */
>-		private static boolean isAllOperandsHaveSameType(InfixExpression expression) {
>-			ITypeBinding binding= expression.getLeftOperand().resolveTypeBinding();
>-			if (binding == null)
>-				return false;
>-
>-			ITypeBinding current= expression.getRightOperand().resolveTypeBinding();
>-			if (binding != current)
>-				return false;
>-
>-			for (Iterator iterator= expression.extendedOperands().iterator(); iterator.hasNext();) {
>-				Expression operand= (Expression) iterator.next();
>-				current= operand.resolveTypeBinding();
>-				if (binding != current)
>-					return false;
>-			}
>-
>-			return true;
>-		}
> 	}
> 
> 	private static class AddParenthesisOperation extends CompilationUnitRewriteOperation {
>@@ -385,5 +224,4 @@
> 	protected ExpressionsFix(String name, CompilationUnit compilationUnit, CompilationUnitRewriteOperation[] fixRewriteOperations) {
> 		super(name, compilationUnit, fixRewriteOperations);
> 	}
>-
> }
>Index: core extension/org/eclipse/jdt/internal/corext/fix/UnusedCodeFix.java
>===================================================================
>RCS file: /cvsroot/eclipse/org.eclipse.jdt.ui/core extension/org/eclipse/jdt/internal/corext/fix/UnusedCodeFix.java,v
>retrieving revision 1.43
>diff -u -r1.43 UnusedCodeFix.java
>--- core extension/org/eclipse/jdt/internal/corext/fix/UnusedCodeFix.java	27 Oct 2010 15:04:41 -0000	1.43
>+++ core extension/org/eclipse/jdt/internal/corext/fix/UnusedCodeFix.java	21 Feb 2011 04:10:09 -0000
>@@ -1,5 +1,5 @@
> /*******************************************************************************
>- * Copyright (c) 2000, 2010 IBM Corporation and others.
>+ * Copyright (c) 2000, 2011 IBM Corporation and others.
> * All rights reserved. This program and the accompanying materials
> * are made available under the terms of the Eclipse Public License v1.0
> * which accompanies this distribution, and is available at
>@@ -64,6 +64,7 @@
> 
> import org.eclipse.jdt.internal.corext.dom.ASTNodes;
> import org.eclipse.jdt.internal.corext.dom.LinkedNodeFinder;
>+import org.eclipse.jdt.internal.corext.dom.NecessaryParenthesesChecker;
> import org.eclipse.jdt.internal.corext.refactoring.structure.CompilationUnitRewrite;
> import org.eclipse.jdt.internal.corext.util.Messages;
> 
>@@ -435,10 +436,10 @@
> 			Expression expression= cast.getExpression();
> 			ASTNode placeholder= rewrite.createCopyTarget(expression);
> 
>-			if (ASTNodes.needsParentheses(expression)) {
>+			if (NecessaryParenthesesChecker.needsParentheses(expression, cast.getParent(), cast.getLocationInParent())) {
> 				rewrite.replace(fCast, placeholder, group);
> 			} else {
>-				rewrite.replace(fSelectedNode, placeholder, group);
>+				rewrite.replace(fCast.getParent() instanceof ParenthesizedExpression ? fCast.getParent() : fSelectedNode, placeholder, group);
> 			}
> 		}
> 	}
>@@ -468,7 +469,8 @@
> 					fUnnecessaryCasts.remove(down);
> 				}
> 
>-				ASTNode move= rewrite.createMoveTarget(down.getExpression());
>+				Expression expression= down.getExpression();
>+				ASTNode move= rewrite.createMoveTarget(expression);
> 
> 				CastExpression top= castExpression;
> 				while (fUnnecessaryCasts.contains(top.getParent())) {
>@@ -476,7 +478,11 @@
> 					fUnnecessaryCasts.remove(top);
> 				}
> 
>-				rewrite.replace(top, move, group);
>+				ASTNode toReplace= top;
>+				if (top.getParent() instanceof ParenthesizedExpression && !NecessaryParenthesesChecker.needsParentheses(expression, top.getParent(), top.getLocationInParent())) {
>+					toReplace= top.getParent();
>+				}
>+				rewrite.replace(toReplace, move, group);
> 			}
> 		}
> 	}
>Index: core refactoring/org/eclipse/jdt/internal/corext/refactoring/code/CallInliner.java
>===================================================================
>RCS file: /cvsroot/eclipse/org.eclipse.jdt.ui/core refactoring/org/eclipse/jdt/internal/corext/refactoring/code/CallInliner.java,v
>retrieving revision 1.89
>diff -u -r1.89 CallInliner.java
>--- core refactoring/org/eclipse/jdt/internal/corext/refactoring/code/CallInliner.java	8 Aug 2010 12:05:48 -0000	1.89
>+++ core refactoring/org/eclipse/jdt/internal/corext/refactoring/code/CallInliner.java	21 Feb 2011 04:10:09 -0000
>@@ -1,5 +1,5 @@
> /*******************************************************************************
>- * Copyright (c) 2000, 2010 IBM Corporation and others.
>+ * Copyright (c) 2000, 2011 IBM Corporation and others.
> * All rights reserved. This program and the accompanying materials
> * are made available under the terms of the Eclipse Public License v1.0
> * which accompanies this distribution, and is available at
>@@ -646,7 +646,7 @@
> 					node= castExpression;
> 				}
> 
>-				if (needsParenthesis()) {
>+				if (fSourceProvider.needsReturnedExpressionParenthesis(fTargetNode.getParent(), fTargetNode.getLocationInParent())) {
> 					ParenthesizedExpression pExp= fTargetNode.getAST().newParenthesizedExpression();
> 					pExp.setExpression((Expression)node);
> 					node= pExp;
>@@ -721,19 +721,6 @@
> 		return false;
> 	}
> 
>-	private boolean needsParenthesis() {
>-		if (!fSourceProvider.needsReturnedExpressionParenthesis())
>-			return false;
>-		ASTNode parent= fTargetNode.getParent();
>-		int type= parent.getNodeType();
>-		return
>-			type == ASTNode.METHOD_INVOCATION ||
>-			(parent instanceof Expression && type != ASTNode.ASSIGNMENT) ||
>-			(fSourceProvider.returnsConditionalExpression() &&
>-				type == ASTNode.VARIABLE_DECLARATION_FRAGMENT &&
>-				((VariableDeclarationFragment)parent).getInitializer() == fTargetNode);
>-	}
>-
> 	private VariableDeclarationStatement createLocalDeclaration(ITypeBinding type, String name, Expression initializer) {
> 		ImportRewriteContext context= new ContextSensitiveImportRewriteContext(fTargetNode, fImportRewrite);
> 		String typeName= fImportRewrite.addImport(type, context);
>Index: core refactoring/org/eclipse/jdt/internal/corext/refactoring/code/ExtractConstantRefactoring.java
>===================================================================
>RCS file: /cvsroot/eclipse/org.eclipse.jdt.ui/core refactoring/org/eclipse/jdt/internal/corext/refactoring/code/ExtractConstantRefactoring.java,v
>retrieving revision 1.128
>diff -u -r1.128 ExtractConstantRefactoring.java
>--- core refactoring/org/eclipse/jdt/internal/corext/refactoring/code/ExtractConstantRefactoring.java	24 Jun 2010 12:21:47 -0000	1.128
>+++ core refactoring/org/eclipse/jdt/internal/corext/refactoring/code/ExtractConstantRefactoring.java	21 Feb 2011 04:10:09 -0000
>@@ -1,5 +1,5 @@
> /*******************************************************************************
>- * Copyright (c) 2000, 2010 IBM Corporation and others.
>+ * Copyright (c) 2000, 2011 IBM Corporation and others.
> * All rights reserved. This program and the accompanying materials
> * are made available under the terms of the Eclipse Public License v1.0
> * which accompanies this distribution, and is available at
>@@ -492,12 +492,12 @@
> 		Type type= getConstantType();
> 
> 		IExpressionFragment fragment= getSelectedExpression();
>-		String initializerSource= fCu.getBuffer().getText(fragment.getStartPosition(), fragment.getLength());
>+		Expression initializer= getSelectedExpression().createCopyTarget(fCuRewrite.getASTRewrite(), true);
> 
> 		AST ast= fCuRewrite.getAST();
> 		VariableDeclarationFragment variableDeclarationFragment= ast.newVariableDeclarationFragment();
> 		variableDeclarationFragment.setName(ast.newSimpleName(fConstantName));
>-		variableDeclarationFragment.setInitializer((Expression) fCuRewrite.getASTRewrite().createStringPlaceholder(initializerSource, ASTNode.SIMPLE_NAME));
>+		variableDeclarationFragment.setInitializer(initializer);
> 
> 		FieldDeclaration fieldDeclaration= ast.newFieldDeclaration(variableDeclarationFragment);
> 		fieldDeclaration.setType(type);
>Index: core refactoring/org/eclipse/jdt/internal/corext/refactoring/code/ExtractMethodRefactoring.java
>===================================================================
>RCS file: /cvsroot/eclipse/org.eclipse.jdt.ui/core refactoring/org/eclipse/jdt/internal/corext/refactoring/code/ExtractMethodRefactoring.java,v
>retrieving revision 1.172
>diff -u -r1.172 ExtractMethodRefactoring.java
>--- core refactoring/org/eclipse/jdt/internal/corext/refactoring/code/ExtractMethodRefactoring.java	9 Nov 2010 04:55:02 -0000	1.172
>+++ core refactoring/org/eclipse/jdt/internal/corext/refactoring/code/ExtractMethodRefactoring.java	21 Feb 2011 04:10:10 -0000
>@@ -1,5 +1,5 @@
> /*******************************************************************************
>- * Copyright (c) 2000, 2010 IBM Corporation and others.
>+ * Copyright (c) 2000, 2011 IBM Corporation and others.
> * All rights reserved. This program and the accompanying materials
> * are made available under the terms of the Eclipse Public License v1.0
> * which accompanies this distribution, and is available at
>@@ -75,6 +75,7 @@
> import org.eclipse.jdt.core.dom.MethodDeclaration;
> import org.eclipse.jdt.core.dom.MethodInvocation;
> import org.eclipse.jdt.core.dom.Modifier;
>+import org.eclipse.jdt.core.dom.ParenthesizedExpression;
> import org.eclipse.jdt.core.dom.QualifiedName;
> import org.eclipse.jdt.core.dom.ReturnStatement;
> import org.eclipse.jdt.core.dom.SimpleName;
>@@ -909,7 +910,14 @@
> 			if (!duplicate.isMethodBody()) {
> 				if (isDestinationReachable(duplicate.getEnclosingMethod())) {
> 					ASTNode[] callNodes= createCallNodes(duplicate, modifiers);
>-					new StatementRewrite(fRewriter, duplicate.getNodes()).replace(callNodes, description);
>+					ASTNode[] duplicateNodes= duplicate.getNodes();
>+					for (int i= 0; i < duplicateNodes.length; i++) {
>+						ASTNode parent= duplicateNodes[i].getParent();
>+						if (parent instanceof ParenthesizedExpression) {
>+							duplicateNodes[i]= parent;
>+						}
>+					}
>+					new StatementRewrite(fRewriter, duplicateNodes).replace(callNodes, description);
> 				}
> 			}
> 		}
>@@ -1059,13 +1067,15 @@
> 			ITypeBinding binding= fAnalyzer.getExpressionBinding();
> 			if (binding != null && (!binding.isPrimitive() || !"void".equals(binding.getName()))) { //$NON-NLS-1$
> 				ReturnStatement rs= fAST.newReturnStatement();
>-				rs.setExpression((Expression)fRewriter.createMoveTarget(selectedNodes[0]));
>+				rs.setExpression((Expression)fRewriter.createMoveTarget(selectedNodes[0] instanceof ParenthesizedExpression
>+						? ((ParenthesizedExpression)selectedNodes[0]).getExpression()
>+						: selectedNodes[0]));
> 				statements.insertLast(rs, null);
> 			} else {
> 				ExpressionStatement st= fAST.newExpressionStatement((Expression)fRewriter.createMoveTarget(selectedNodes[0]));
> 				statements.insertLast(st, null);
> 			}
>-			fRewriter.replace(selectedNodes[0], replacementNode, substitute);
>+			fRewriter.replace(selectedNodes[0].getParent() instanceof ParenthesizedExpression ? selectedNodes[0].getParent() : selectedNodes[0], replacementNode, substitute);
> 		} else {
> 			if (selectedNodes.length == 1) {
> 				statements.insertLast(fRewriter.createMoveTarget(selectedNodes[0]), substitute);
>Index: core refactoring/org/eclipse/jdt/internal/corext/refactoring/code/InlineConstantRefactoring.java
>===================================================================
>RCS file: /cvsroot/eclipse/org.eclipse.jdt.ui/core refactoring/org/eclipse/jdt/internal/corext/refactoring/code/InlineConstantRefactoring.java,v
>retrieving revision 1.120
>diff -u -r1.120 InlineConstantRefactoring.java
>--- core refactoring/org/eclipse/jdt/internal/corext/refactoring/code/InlineConstantRefactoring.java	18 Feb 2010 21:25:28 -0000	1.120
>+++ core refactoring/org/eclipse/jdt/internal/corext/refactoring/code/InlineConstantRefactoring.java	21 Feb 2011 04:10:10 -0000
>@@ -1,5 +1,5 @@
> /*******************************************************************************
>- * Copyright (c) 2000, 2010 IBM Corporation and others.
>+ * Copyright (c) 2000, 2011 IBM Corporation and others.
> * All rights reserved. This program and the accompanying materials
> * are made available under the terms of the Eclipse Public License v1.0
> * which accompanies this distribution, and is available at
>@@ -83,8 +83,8 @@
> import org.eclipse.jdt.core.dom.VariableDeclaration;
> import org.eclipse.jdt.core.dom.VariableDeclarationFragment;
> import org.eclipse.jdt.core.dom.rewrite.ASTRewrite;
>-import org.eclipse.jdt.core.dom.rewrite.ListRewrite;
> import org.eclipse.jdt.core.dom.rewrite.ImportRewrite.ImportRewriteContext;
>+import org.eclipse.jdt.core.dom.rewrite.ListRewrite;
> import org.eclipse.jdt.core.refactoring.CompilationUnitChange;
> import org.eclipse.jdt.core.refactoring.IJavaRefactorings;
> import org.eclipse.jdt.core.refactoring.descriptors.InlineConstantDescriptor;
>@@ -100,6 +100,7 @@
> import org.eclipse.jdt.internal.corext.dom.ASTNodeFactory;
> import org.eclipse.jdt.internal.corext.dom.ASTNodes;
> import org.eclipse.jdt.internal.corext.dom.HierarchicalASTVisitor;
>+import org.eclipse.jdt.internal.corext.dom.NecessaryParenthesesChecker;
> import org.eclipse.jdt.internal.corext.dom.fragments.ASTFragmentFactory;
> import org.eclipse.jdt.internal.corext.dom.fragments.IExpressionFragment;
> import org.eclipse.jdt.internal.corext.refactoring.Checks;
>@@ -501,7 +502,7 @@
> 			if (explicitCast != null) {
> 				CastExpression cast= ast.newCastExpression();
> 				Expression modifiedInitializerExpr= (Expression) fCuRewrite.getASTRewrite().createStringPlaceholder(modifiedInitializer, reference.getNodeType());
>-				if (ASTNodes.substituteMustBeParenthesized(fInitializer, cast)) {
>+				if (NecessaryParenthesesChecker.needsParentheses(fInitializer, cast, CastExpression.EXPRESSION_PROPERTY)) {
> 					ParenthesizedExpression parenthesized= ast.newParenthesizedExpression();
> 					parenthesized.setExpression(modifiedInitializerExpr);
> 					modifiedInitializerExpr= parenthesized;
>@@ -531,7 +532,7 @@
> 				isStringPlaceholder= true;
> 			}
> 
>-			if (ASTNodes.substituteMustBeParenthesized((isStringPlaceholder ? fInitializer : newReference), reference)) {
>+			if (NecessaryParenthesesChecker.needsParentheses((isStringPlaceholder ? fInitializer : newReference), reference.getParent(), reference.getLocationInParent())) {
> 				ParenthesizedExpression parenthesized= ast.newParenthesizedExpression();
> 				parenthesized.setExpression(newReference);
> 				newReference= parenthesized;
>Index: core refactoring/org/eclipse/jdt/internal/corext/refactoring/code/InlineTempRefactoring.java
>===================================================================
>RCS file: /cvsroot/eclipse/org.eclipse.jdt.ui/core refactoring/org/eclipse/jdt/internal/corext/refactoring/code/InlineTempRefactoring.java,v
>retrieving revision 1.112
>diff -u -r1.112 InlineTempRefactoring.java
>--- core refactoring/org/eclipse/jdt/internal/corext/refactoring/code/InlineTempRefactoring.java	19 Jan 2010 14:09:11 -0000	1.112
>+++ core refactoring/org/eclipse/jdt/internal/corext/refactoring/code/InlineTempRefactoring.java	21 Feb 2011 04:10:10 -0000
>@@ -1,5 +1,5 @@
> /*******************************************************************************
>- * Copyright (c) 2000, 2010 IBM Corporation and others.
>+ * Copyright (c) 2000, 2011 IBM Corporation and others.
> * All rights reserved. This program and the accompanying materials
> * are made available under the terms of the Eclipse Public License v1.0
> * which accompanies this distribution, and is available at
>@@ -79,6 +79,7 @@
> import org.eclipse.jdt.internal.core.refactoring.descriptors.RefactoringSignatureDescriptorFactory;
> import org.eclipse.jdt.internal.corext.dom.ASTNodeFactory;
> import org.eclipse.jdt.internal.corext.dom.ASTNodes;
>+import org.eclipse.jdt.internal.corext.dom.NecessaryParenthesesChecker;
> import org.eclipse.jdt.internal.corext.refactoring.Checks;
> import org.eclipse.jdt.internal.corext.refactoring.JDTRefactoringDescriptorComment;
> import org.eclipse.jdt.internal.corext.refactoring.JavaRefactoringArguments;
>@@ -333,8 +334,7 @@
> 
> 	private Expression getInitializerSource(CompilationUnitRewrite rewrite, SimpleName reference) throws JavaModelException {
> 		Expression copy= getModifiedInitializerSource(rewrite, reference);
>-		boolean brackets= ASTNodes.substituteMustBeParenthesized(copy, reference);
>-		if (brackets) {
>+		if (NecessaryParenthesesChecker.needsParentheses(copy, reference.getParent(), reference.getLocationInParent())) {
> 			ParenthesizedExpression parentExpr= rewrite.getAST().newParenthesizedExpression();
> 			parentExpr.setExpression(copy);
> 			return parentExpr;
>@@ -367,7 +367,7 @@
> 		ITypeBinding explicitCast= ASTNodes.getExplicitCast(initializer, reference);
> 		if (explicitCast != null) {
> 			CastExpression cast= ast.newCastExpression();
>-			if (ASTNodes.substituteMustBeParenthesized(copy, cast)) {
>+			if (NecessaryParenthesesChecker.needsParentheses(initializer, cast, CastExpression.EXPRESSION_PROPERTY)) {
> 				ParenthesizedExpression parenthesized= ast.newParenthesizedExpression();
> 				parenthesized.setExpression(copy);
> 				copy= parenthesized;
>Index: core refactoring/org/eclipse/jdt/internal/corext/refactoring/code/IntroduceParameterRefactoring.java
>===================================================================
>RCS file: /cvsroot/eclipse/org.eclipse.jdt.ui/core refactoring/org/eclipse/jdt/internal/corext/refactoring/code/IntroduceParameterRefactoring.java,v
>retrieving revision 1.91
>diff -u -r1.91 IntroduceParameterRefactoring.java
>--- core refactoring/org/eclipse/jdt/internal/corext/refactoring/code/IntroduceParameterRefactoring.java	25 Aug 2010 10:24:03 -0000	1.91
>+++ core refactoring/org/eclipse/jdt/internal/corext/refactoring/code/IntroduceParameterRefactoring.java	21 Feb 2011 04:10:10 -0000
>@@ -1,5 +1,5 @@
> /*******************************************************************************
>- * Copyright (c) 2000, 2010 IBM Corporation and others.
>+ * Copyright (c) 2000, 2011 IBM Corporation and others.
> * All rights reserved. This program and the accompanying materials
> * are made available under the terms of the Eclipse Public License v1.0
> * which accompanies this distribution, and is available at
>@@ -55,6 +55,7 @@
> import org.eclipse.jdt.core.dom.Name;
> import org.eclipse.jdt.core.dom.NodeFinder;
> import org.eclipse.jdt.core.dom.NullLiteral;
>+import org.eclipse.jdt.core.dom.ParenthesizedExpression;
> import org.eclipse.jdt.core.dom.QualifiedName;
> import org.eclipse.jdt.core.dom.SimpleName;
> import org.eclipse.jdt.core.refactoring.IJavaRefactorings;
>@@ -254,7 +255,8 @@
> 		ITypeBinding typeBinding= Bindings.normalizeForDeclarationUse(fSelectedExpression.resolveTypeBinding(), fSelectedExpression.getAST());
> 		String typeName= cuRewrite.getImportRewrite().addImport(typeBinding);
> 		String name= fParameterName != null ? fParameterName : guessedParameterName();
>-		String defaultValue= fSourceCU.getBuffer().getText(fSelectedExpression.getStartPosition(), fSelectedExpression.getLength());
>+		Expression expression= fSelectedExpression instanceof ParenthesizedExpression ? ((ParenthesizedExpression)fSelectedExpression).getExpression() : fSelectedExpression;
>+		String defaultValue= fSourceCU.getBuffer().getText(expression.getStartPosition(), expression.getLength());
> 		fParameter= ParameterInfo.createInfoForAddedParameter(typeBinding, typeName, name, defaultValue);
> 		if (fArguments == null) {
> 			List parameterInfos= fChangeSignatureProcessor.getParameterInfos();
>@@ -276,7 +278,8 @@
> 
> 		ASTNode newExpression= cuRewrite.getRoot().getAST().newSimpleName(fParameter.getNewName());
> 		String description= RefactoringCoreMessages.IntroduceParameterRefactoring_replace;
>-		cuRewrite.getASTRewrite().replace(expression, newExpression, cuRewrite.createGroupDescription(description));
>+		cuRewrite.getASTRewrite().replace(expression.getParent() instanceof ParenthesizedExpression
>+				? expression.getParent() : expression, newExpression, cuRewrite.createGroupDescription(description));
> 	}
> 
> 	private void initializeSelectedExpression(CompilationUnitRewrite cuRewrite) throws JavaModelException {
>Index: core refactoring/org/eclipse/jdt/internal/corext/refactoring/code/SourceProvider.java
>===================================================================
>RCS file: /cvsroot/eclipse/org.eclipse.jdt.ui/core refactoring/org/eclipse/jdt/internal/corext/refactoring/code/SourceProvider.java,v
>retrieving revision 1.71
>diff -u -r1.71 SourceProvider.java
>--- core refactoring/org/eclipse/jdt/internal/corext/refactoring/code/SourceProvider.java	17 Nov 2010 05:26:24 -0000	1.71
>+++ core refactoring/org/eclipse/jdt/internal/corext/refactoring/code/SourceProvider.java	21 Feb 2011 04:10:10 -0000
>@@ -1,5 +1,5 @@
> /*******************************************************************************
>- * Copyright (c) 2000, 2010 IBM Corporation and others.
>+ * Copyright (c) 2000, 2011 IBM Corporation and others.
> * All rights reserved. This program and the accompanying materials
> * are made available under the terms of the Eclipse Public License v1.0
> * which accompanies this distribution, and is available at
>@@ -48,7 +48,6 @@
> import org.eclipse.jdt.core.dom.AST;
> import org.eclipse.jdt.core.dom.ASTNode;
> import org.eclipse.jdt.core.dom.ASTVisitor;
>-import org.eclipse.jdt.core.dom.ArrayCreation;
> import org.eclipse.jdt.core.dom.Block;
> import org.eclipse.jdt.core.dom.CastExpression;
> import org.eclipse.jdt.core.dom.ChildPropertyDescriptor;
>@@ -75,6 +74,7 @@
> import org.eclipse.jdt.core.dom.SimpleName;
> import org.eclipse.jdt.core.dom.SingleVariableDeclaration;
> import org.eclipse.jdt.core.dom.Statement;
>+import org.eclipse.jdt.core.dom.StructuralPropertyDescriptor;
> import org.eclipse.jdt.core.dom.ThisExpression;
> import org.eclipse.jdt.core.dom.WhileStatement;
> import org.eclipse.jdt.core.dom.rewrite.ASTRewrite;
>@@ -85,6 +85,7 @@
> import org.eclipse.jdt.internal.corext.dom.ASTNodes;
> import org.eclipse.jdt.internal.corext.dom.Bindings;
> import org.eclipse.jdt.internal.corext.dom.CodeScopeBuilder;
>+import org.eclipse.jdt.internal.corext.dom.NecessaryParenthesesChecker;
> import org.eclipse.jdt.internal.corext.refactoring.code.SourceAnalyzer.NameData;
> import org.eclipse.jdt.internal.corext.refactoring.util.RefactoringFileBuffers;
> import org.eclipse.jdt.internal.corext.util.CodeFormatterUtil;
>@@ -314,10 +315,10 @@
> 		return fTypeRoot;
> 	}
> 
>-	public boolean needsReturnedExpressionParenthesis() {
>+	public boolean needsReturnedExpressionParenthesis(ASTNode parent, StructuralPropertyDescriptor locationInParent) {
> 		ASTNode last= getLastStatement();
> 		if (last instanceof ReturnStatement) {
>-			return ASTNodes.needsParentheses(((ReturnStatement)last).getExpression());
>+			return NecessaryParenthesesChecker.needsParentheses(((ReturnStatement)last).getExpression(), parent, locationInParent);
> 		}
> 		return false;
> 	}
>@@ -413,16 +414,6 @@
> 		return new String[] {};
> 	}
> 
>-	private boolean argumentNeedsParenthesis(Expression expression, ParameterData param) {
>-		if (expression instanceof CastExpression || expression instanceof ArrayCreation)
>-			return true;
>-		int argPrecedence= OperatorPrecedence.getExpressionPrecedence(expression);
>-		int paramPrecedence= param.getOperatorPrecedence();
>-		if (argPrecedence != Integer.MAX_VALUE && paramPrecedence != Integer.MAX_VALUE)
>-			return argPrecedence <= paramPrecedence;
>-		return false;
>-	}
>-
> 	private Expression createParenthesizedExpression(Expression newExpression, AST ast) {
> 		ParenthesizedExpression parenthesized= ast.newParenthesizedExpression();
> 		parenthesized.setExpression(newExpression);
>@@ -455,14 +446,15 @@
> 					ITypeBinding explicitCast= ASTNodes.getExplicitCast(expression, (Expression)element);
> 					if (explicitCast != null) {
> 						CastExpression cast= ast.newCastExpression();
>-						if (ASTNodes.substituteMustBeParenthesized(newExpression, cast)) {
>+						if (NecessaryParenthesesChecker.needsParentheses(expression, cast, CastExpression.EXPRESSION_PROPERTY)) {
> 							newExpression= createParenthesizedExpression(newExpression, ast);
> 						}
> 						cast.setExpression(newExpression);
> 						ImportRewriteContext importRewriteContext= new ContextSensitiveImportRewriteContext(expression, importRewrite);
> 						cast.setType(importRewrite.addImport(explicitCast, ast, importRewriteContext));
>-						newExpression= createParenthesizedExpression(cast, ast);
>-					} else if (argumentNeedsParenthesis(expression, parameter)) {
>+						expression= newExpression= cast;
>+					}
>+					if (NecessaryParenthesesChecker.needsParentheses(expression, element.getParent(), element.getLocationInParent())) {
> 						newExpression= createParenthesizedExpression(newExpression, ast);
> 					}
> 					rewriter.replace(element, newExpression, null);
>Index: core refactoring/org/eclipse/jdt/internal/corext/refactoring/sef/AccessAnalyzer.java
>===================================================================
>RCS file: /cvsroot/eclipse/org.eclipse.jdt.ui/core refactoring/org/eclipse/jdt/internal/corext/refactoring/sef/AccessAnalyzer.java,v
>retrieving revision 1.41
>diff -u -r1.41 AccessAnalyzer.java
>--- core refactoring/org/eclipse/jdt/internal/corext/refactoring/sef/AccessAnalyzer.java	21 May 2010 07:28:38 -0000	1.41
>+++ core refactoring/org/eclipse/jdt/internal/corext/refactoring/sef/AccessAnalyzer.java	21 Feb 2011 04:10:10 -0000
>@@ -1,5 +1,5 @@
> /*******************************************************************************
>- * Copyright (c) 2000, 2009 IBM Corporation and others.
>+ * Copyright (c) 2000, 2011 IBM Corporation and others.
> * All rights reserved. This program and the accompanying materials
> * are made available under the terms of the Eclipse Public License v1.0
> * which accompanies this distribution, and is available at
>@@ -51,6 +51,7 @@
> import org.eclipse.jdt.internal.corext.SourceRangeFactory;
> import org.eclipse.jdt.internal.corext.dom.ASTNodes;
> import org.eclipse.jdt.internal.corext.dom.Bindings;
>+import org.eclipse.jdt.internal.corext.dom.NecessaryParenthesesChecker;
> import org.eclipse.jdt.internal.corext.refactoring.RefactoringCoreMessages;
> import org.eclipse.jdt.internal.corext.refactoring.base.JavaStatusContext;
> 
>@@ -138,7 +139,6 @@
> 				arguments.add(fRewriter.createCopyTarget(node.getRightHandSide()));
> 			} else {
> 				// This is the compound assignment case: field+= 10;
>-				boolean needsParentheses= ASTNodes.needsParentheses(node.getRightHandSide());
> 				InfixExpression exp= ast.newInfixExpression();
> 				exp.setOperator(ASTNodes.convertToInfixOperator(node.getOperator()));
> 				MethodInvocation getter= ast.newMethodInvocation();
>@@ -148,7 +148,8 @@
> 					getter.setExpression((Expression)fRewriter.createCopyTarget(receiver));
> 				exp.setLeftOperand(getter);
> 				Expression rhs= (Expression)fRewriter.createCopyTarget(node.getRightHandSide());
>-				if (needsParentheses) {
>+				if (NecessaryParenthesesChecker.needsParentheses(node.getRightHandSide(), exp, InfixExpression.RIGHT_OPERAND_PROPERTY)) {
>+					//TODO: this introduces extra parentheses as the new 'exp' node doesn't have bindings
> 					ParenthesizedExpression p= ast.newParenthesizedExpression();
> 					p.setExpression(rhs);
> 					rhs= p;
>Index: ui/org/eclipse/jdt/internal/ui/text/correction/AdvancedQuickAssistProcessor.java
>===================================================================
>RCS file: /cvsroot/eclipse/org.eclipse.jdt.ui/ui/org/eclipse/jdt/internal/ui/text/correction/AdvancedQuickAssistProcessor.java,v
>retrieving revision 1.81
>diff -u -r1.81 AdvancedQuickAssistProcessor.java
>--- ui/org/eclipse/jdt/internal/ui/text/correction/AdvancedQuickAssistProcessor.java	3 Feb 2011 14:49:29 -0000	1.81
>+++ ui/org/eclipse/jdt/internal/ui/text/correction/AdvancedQuickAssistProcessor.java	21 Feb 2011 04:10:10 -0000
>@@ -84,6 +84,7 @@
> import org.eclipse.jdt.internal.corext.dom.ASTNodes;
> import org.eclipse.jdt.internal.corext.dom.GenericVisitor;
> import org.eclipse.jdt.internal.corext.dom.LinkedNodeFinder;
>+import org.eclipse.jdt.internal.corext.dom.NecessaryParenthesesChecker;
> import org.eclipse.jdt.internal.corext.fix.CleanUpConstants;
> import org.eclipse.jdt.internal.corext.fix.ExpressionsFix;
> import org.eclipse.jdt.internal.corext.fix.IProposableFix;
>@@ -523,7 +524,12 @@
> 		if (expression instanceof PrefixExpression) {
> 			PrefixExpression prefixExpression= (PrefixExpression) expression;
> 			if (prefixExpression.getOperator() == PrefixExpression.Operator.NOT) {
>-				return getRenamedNameCopy(provider, rewrite, prefixExpression.getOperand());
>+				Expression operand= prefixExpression.getOperand();
>+				if ((operand instanceof ParenthesizedExpression)
>+						&& NecessaryParenthesesChecker.canRemoveParentheses(operand, expression.getParent(), expression.getLocationInParent())) {
>+					operand= ((ParenthesizedExpression)operand).getExpression();
>+				}
>+				return getRenamedNameCopy(provider, rewrite, operand);
> 			}
> 		}
> 		if (expression instanceof InstanceofExpression) {
>@@ -745,12 +751,12 @@
> 				//
> 				AST ast= node.getAST();
> 				ASTRewrite rewrite= ASTRewrite.create(ast);
>-				// prepare condition parts, add parentheses if needed
>-				Expression outerCondition= getParenthesizedForAndIfNeeded(ast, rewrite, outerIf.getExpression());
>-				Expression innerCondition= getParenthesizedForAndIfNeeded(ast, rewrite, ifStatement.getExpression());
> 				// create compound condition
> 				InfixExpression condition= ast.newInfixExpression();
> 				condition.setOperator(InfixExpression.Operator.CONDITIONAL_AND);
>+				// prepare condition parts, add parentheses if needed
>+				Expression outerCondition= getParenthesizedExpressionIfNeeded(ast, rewrite, outerIf.getExpression(), condition, InfixExpression.LEFT_OPERAND_PROPERTY);
>+				Expression innerCondition= getParenthesizedExpressionIfNeeded(ast, rewrite, ifStatement.getExpression(), condition, InfixExpression.RIGHT_OPERAND_PROPERTY);
> 				condition.setLeftOperand(outerCondition);
> 				condition.setRightOperand(innerCondition);
> 				// create new IfStatement
>@@ -784,12 +790,12 @@
> 				//
> 				AST ast= node.getAST();
> 				ASTRewrite rewrite= ASTRewrite.create(ast);
>-				// prepare condition parts, add parentheses if needed
>-				Expression outerCondition= getParenthesizedForAndIfNeeded(ast, rewrite, ifStatement.getExpression());
>-				Expression innerCondition= getParenthesizedForAndIfNeeded(ast, rewrite, innerIf.getExpression());
> 				// create compound condition
> 				InfixExpression condition= ast.newInfixExpression();
> 				condition.setOperator(InfixExpression.Operator.CONDITIONAL_AND);
>+				// prepare condition parts, add parentheses if needed
>+				Expression outerCondition= getParenthesizedExpressionIfNeeded(ast, rewrite, ifStatement.getExpression(), condition, InfixExpression.LEFT_OPERAND_PROPERTY);
>+				Expression innerCondition= getParenthesizedExpressionIfNeeded(ast, rewrite, innerIf.getExpression(), condition, InfixExpression.RIGHT_OPERAND_PROPERTY);
> 				condition.setLeftOperand(outerCondition);
> 				condition.setRightOperand(innerCondition);
> 				// create new IfStatement
>@@ -808,16 +814,9 @@
> 		return true;
> 	}
> 
>-	private static Expression getParenthesizedForAndIfNeeded(AST ast, ASTRewrite rewrite, Expression expression) {
>-		boolean addParentheses= false;
>-		int nodeType= expression.getNodeType();
>-		if (nodeType == ASTNode.INFIX_EXPRESSION) {
>-			InfixExpression infixExpression= (InfixExpression) expression;
>-			addParentheses= infixExpression.getOperator() == InfixExpression.Operator.CONDITIONAL_OR;
>-		} else {
>-			addParentheses= nodeType == ASTNode.CONDITIONAL_EXPRESSION || nodeType == ASTNode.ASSIGNMENT || nodeType == ASTNode.INSTANCEOF_EXPRESSION;
>-		}
>-		expression= (Expression) rewrite.createCopyTarget(expression);
>+	private static Expression getParenthesizedExpressionIfNeeded(AST ast, ASTRewrite rewrite, Expression expression, ASTNode parent, StructuralPropertyDescriptor locationInParent) {
>+		boolean addParentheses= NecessaryParenthesesChecker.needsParentheses(expression, parent, locationInParent);
>+		expression= (Expression)rewrite.createCopyTarget(expression);
> 		if (addParentheses) {
> 			return getParenthesizedExpression(ast, expression);
> 		}
>@@ -976,19 +975,18 @@
> 			IfStatement ifStatement= (IfStatement) iter.next();
> 			if (thenStatement == null)
> 				thenStatement= (Statement) rewrite.createCopyTarget(ifStatement.getThenStatement());
>-			Expression ifCondition= getParenthesizedForOrIfNeeded(ast, rewrite, ifStatement.getExpression());
> 			if (condition == null) {
> 				condition= ast.newInfixExpression();
> 				condition.setOperator(orOperator);
>-				condition.setLeftOperand(ifCondition);
>+				condition.setLeftOperand(getParenthesizedExpressionIfNeeded(ast, rewrite, ifStatement.getExpression(), condition, InfixExpression.LEFT_OPERAND_PROPERTY));
> 			} else if (!hasRightOperand) {
>-				condition.setRightOperand(ifCondition);
>+				condition.setRightOperand(getParenthesizedExpressionIfNeeded(ast, rewrite, ifStatement.getExpression(), condition, InfixExpression.RIGHT_OPERAND_PROPERTY));
> 				hasRightOperand= true;
> 			} else {
> 				InfixExpression newCondition= ast.newInfixExpression();
> 				newCondition.setOperator(orOperator);
> 				newCondition.setLeftOperand(condition);
>-				newCondition.setRightOperand(ifCondition);
>+				newCondition.setRightOperand(getParenthesizedExpressionIfNeeded(ast, rewrite, ifStatement.getExpression(), condition, InfixExpression.RIGHT_OPERAND_PROPERTY));
> 				condition= newCondition;
> 			}
> 		}
>@@ -1020,17 +1018,6 @@
> 		return true;
> 	}
> 
>-	private static Expression getParenthesizedForOrIfNeeded(AST ast, ASTRewrite rewrite, Expression expression) {
>-		boolean addParentheses= false;
>-		int nodeType= expression.getNodeType();
>-		addParentheses= nodeType == ASTNode.CONDITIONAL_EXPRESSION || nodeType == ASTNode.ASSIGNMENT || nodeType == ASTNode.INSTANCEOF_EXPRESSION;
>-		expression= (Expression) rewrite.createCopyTarget(expression);
>-		if (addParentheses) {
>-			return getParenthesizedExpression(ast, expression);
>-		}
>-		return expression;
>-	}
>-
> 	public static boolean getSplitOrConditionProposals(IInvocationContext context, ASTNode node, Collection resultingCollections) {
> 		Operator orOperator= InfixExpression.Operator.CONDITIONAL_OR;
> 		// check that user invokes quick assist on infix expression
>@@ -1254,14 +1241,11 @@
> 			}
> 		}
> 
>-		// see https://bugs.eclipse.org/bugs/show_bug.cgi?id=332019
>-		if (operator == InfixExpression.Operator.EQUALS || operator == InfixExpression.Operator.NOT_EQUALS) {
>-			if (leftExpression instanceof InfixExpression && !(leftExpression instanceof ParenthesizedExpression)) {
>-				leftExpression= getParenthesizedExpression(ast, leftExpression);
>-			}
>-			if (rightExpression instanceof InfixExpression && !(rightExpression instanceof ParenthesizedExpression)) {
>-				rightExpression= getParenthesizedExpression(ast, rightExpression);
>-			}
>+		if (NecessaryParenthesesChecker.needsParentheses(leftExpression, infixExpression, InfixExpression.RIGHT_OPERAND_PROPERTY)) {
>+			leftExpression= getParenthesizedExpression(ast, leftExpression);
>+		}
>+		if (NecessaryParenthesesChecker.needsParentheses(rightExpression, infixExpression, InfixExpression.LEFT_OPERAND_PROPERTY)) {
>+			rightExpression= getParenthesizedExpression(ast, rightExpression);
> 		}
> 
> 		if (operator == InfixExpression.Operator.LESS) {
>@@ -1319,13 +1303,17 @@
> 		}
> 	}
> 
>-	private static Expression combineOperands(ASTRewrite rewrite, Expression existing, Expression nodeToAdd, boolean removeParentheses, Operator operator) {
>+	private static Expression combineOperands(ASTRewrite rewrite, Expression existing, Expression originalNode, boolean removeParentheses, Operator operator) {
> 		if (existing == null && removeParentheses) {
>-			while (nodeToAdd instanceof ParenthesizedExpression) {
>-				nodeToAdd= ((ParenthesizedExpression) nodeToAdd).getExpression();
>+			while (originalNode instanceof ParenthesizedExpression) {
>+				originalNode= ((ParenthesizedExpression)originalNode).getExpression();
> 			}
> 		}
>-		Expression newRight= (Expression) rewrite.createMoveTarget(nodeToAdd);
>+		Expression newRight= (Expression)rewrite.createMoveTarget(originalNode);
>+		if (originalNode instanceof InfixExpression) {
>+			((InfixExpression)newRight).setOperator(((InfixExpression)originalNode).getOperator());
>+		}
>+
> 		if (existing == null) {
> 			return newRight;
> 		}
>#P org.eclipse.jdt.ui.tests
>Index: ui/org/eclipse/jdt/ui/tests/quickfix/AdvancedQuickAssistTest.java
>===================================================================
>RCS file: /cvsroot/eclipse/org.eclipse.jdt.ui.tests/ui/org/eclipse/jdt/ui/tests/quickfix/AdvancedQuickAssistTest.java,v
>retrieving revision 1.45
>diff -u -r1.45 AdvancedQuickAssistTest.java
>--- ui/org/eclipse/jdt/ui/tests/quickfix/AdvancedQuickAssistTest.java	29 Jan 2011 05:29:06 -0000	1.45
>+++ ui/org/eclipse/jdt/ui/tests/quickfix/AdvancedQuickAssistTest.java	21 Feb 2011 04:10:13 -0000
>@@ -382,6 +382,40 @@
> 
> 	}
> 
>+	public void testJoinAndIfStatementsBug335173() throws Exception {
>+		IPackageFragment pack1= fSourceFolder.createPackageFragment("test1", false, null);
>+		StringBuffer buf= new StringBuffer();
>+		buf.append("package test1;\n");
>+		buf.append("public class E {\n");
>+		buf.append(" public void foo(Object a, int x) {\n");
>+		buf.append(" if (a instanceof String) {\n");
>+		buf.append(" if (x > 2) {\n");
>+		buf.append(" }\n");
>+		buf.append(" }\n");
>+		buf.append(" }\n");
>+		buf.append("}\n");
>+		ICompilationUnit cu= pack1.createCompilationUnit("E.java", buf.toString(), false, null);
>+
>+		int offset= buf.toString().indexOf("if (a");
>+		AssistContext context= getCorrectionContext(cu, offset, 0);
>+		List proposals= collectAssists(context, false);
>+
>+		assertCorrectLabels(proposals);
>+
>+		buf= new StringBuffer();
>+		buf.append("package test1;\n");
>+		buf.append("public class E {\n");
>+		buf.append(" public void foo(Object a, int x) {\n");
>+		buf.append(" if (a instanceof String && x > 2) {\n");
>+		buf.append(" }\n");
>+		buf.append(" }\n");
>+		buf.append("}\n");
>+		String expected1= buf.toString();
>+
>+		assertExpectedExistInProposals(proposals, new String[] { expected1 });
>+
>+	}
>+
> 	public void testJoinOrIfStatements1() throws Exception {
> 		IPackageFragment pack1= fSourceFolder.createPackageFragment("test1", false, null);
> 		StringBuffer buf= new StringBuffer();
>@@ -425,6 +459,49 @@
> 
> 	}
> 
>+	public void testJoinOrIfStatementsBug335173() throws Exception {
>+		IPackageFragment pack1= fSourceFolder.createPackageFragment("test1", false, null);
>+		StringBuffer buf= new StringBuffer();
>+		buf.append("package test1;\n");
>+		buf.append("public class E {\n");
>+		buf.append(" public void foo(Object a, int x) {\n");
>+		buf.append(" if (a instanceof String)\n");
>+		buf.append(" return;\n");
>+		buf.append(" if (x > 2)\n");
>+		buf.append(" return;\n");
>+		buf.append(" x= 9;\n");
>+		buf.append(" }\n");
>+		buf.append("}\n");
>+		ICompilationUnit cu= pack1.createCompilationUnit("E.java", buf.toString(), false, null);
>+
>+		int offset1= buf.toString().indexOf("if (a");
>+		int offset2= buf.toString().lastIndexOf("x= 9;");
>+		AssistContext context= getCorrectionContext(cu, offset1, offset2 - offset1);
>+		List proposals= collectAssists(context, false);
>+
>+		for (Iterator I= proposals.iterator(); I.hasNext();) {
>+			Object o= I.next();
>+			if (!(o instanceof CUCorrectionProposal))
>+				I.remove();
>+		}
>+
>+		assertCorrectLabels(proposals);
>+
>+		buf= new StringBuffer();
>+		buf.append("package test1;\n");
>+		buf.append("public class E {\n");
>+		buf.append(" public void foo(Object a, int x) {\n");
>+		buf.append(" if (a instanceof String || x > 2)\n");
>+		buf.append(" return;\n");
>+		buf.append(" x= 9;\n");
>+		buf.append(" }\n");
>+		buf.append("}\n");
>+		String expected1= buf.toString();
>+
>+		assertExpectedExistInProposals(proposals, new String[] { expected1 });
>+
>+	}
>+
> 	public void testSplitOrCondition1() throws Exception {
> 		IPackageFragment pack1= fSourceFolder.createPackageFragment("test1", false, null);
> 		StringBuffer buf= new StringBuffer();
>@@ -1639,7 +1716,7 @@
> 		buf.append("package test1;\n");
> 		buf.append("public class E {\n");
> 		buf.append(" public boolean foo(int a, int b) {\n");
>-		buf.append(" return (a == b) != (b > 0);\n");
>+		buf.append(" return (a == b) != b > 0;\n");
> 		buf.append(" }\n");
> 		buf.append("}\n");
> 		String expected1= buf.toString();
>@@ -1679,6 +1756,37 @@
> 
> 	}
> 
>+	public void testExchangeOperandsBug332019_4() throws Exception {
>+		IPackageFragment pack1= fSourceFolder.createPackageFragment("test1", false, null);
>+		StringBuffer buf= new StringBuffer();
>+		buf.append("package test1;\n");
>+		buf.append("public class E {\n");
>+		buf.append(" public boolean foo(int a, int b) {\n");
>+		buf.append(" return b + 1 != a - 1;\n");
>+		buf.append(" }\n");
>+		buf.append("}\n");
>+		ICompilationUnit cu= pack1.createCompilationUnit("E.java", buf.toString(), false, null);
>+
>+		int offset= buf.toString().indexOf("!=");
>+		AssistContext context= getCorrectionContext(cu, offset, 0);
>+		assertNoErrors(context);
>+		List proposals= collectAssists(context, false);
>+
>+		assertCorrectLabels(proposals);
>+
>+		buf= new StringBuffer();
>+		buf.append("package test1;\n");
>+		buf.append("public class E {\n");
>+		buf.append(" public boolean foo(int a, int b) {\n");
>+		buf.append(" return a - 1 != b + 1;\n");
>+		buf.append(" }\n");
>+		buf.append("}\n");
>+		String expected1= buf.toString();
>+
>+		assertExpectedExistInProposals(proposals, new String[] { expected1 });
>+
>+	}
>+
> 	public void testAssignAndCast1() throws Exception {
> 		//https://bugs.eclipse.org/bugs/show_bug.cgi?id=75066
> 		IPackageFragment pack1= fSourceFolder.createPackageFragment("test1", false, null);
>@@ -2895,6 +3003,169 @@
> 
> 	}
> 
>+	public void testInverseCondition2() throws Exception {
>+		IPackageFragment pack1= fSourceFolder.createPackageFragment("test1", false, null);
>+		StringBuffer buf= new StringBuffer();
>+		buf.append("package test1;\n");
>+		buf.append("public class E {\n");
>+		buf.append(" public void foo(Object a) {\n");
>+		buf.append(" if (!(a instanceof String)) {\n");
>+		buf.append(" }\n");
>+		buf.append(" }\n");
>+		buf.append("}\n");
>+		ICompilationUnit cu= pack1.createCompilationUnit("E.java", buf.toString(), false, null);
>+
>+		int offset= buf.toString().indexOf("!");
>+		int length= "!(a instanceof String)".length();
>+		AssistContext context= getCorrectionContext(cu, offset, length);
>+		List proposals= collectAssists(context, false);
>+
>+		assertCorrectLabels(proposals);
>+
>+		buf= new StringBuffer();
>+		buf.append("package test1;\n");
>+		buf.append("public class E {\n");
>+		buf.append(" public void foo(Object a) {\n");
>+		buf.append(" if (a instanceof String) {\n");
>+		buf.append(" }\n");
>+		buf.append(" }\n");
>+		buf.append("}\n");
>+		String expected1= buf.toString();
>+
>+		assertExpectedExistInProposals(proposals, new String[] { expected1 });
>+
>+	}
>+
>+	public void testInverseCondition3() throws Exception {
>+		IPackageFragment pack1= fSourceFolder.createPackageFragment("test1", false, null);
>+		StringBuffer buf= new StringBuffer();
>+		buf.append("package test1;\n");
>+		buf.append("public class E {\n");
>+		buf.append(" public void foo(Object a) {\n");
>+		buf.append(" while (!(a instanceof String)) {\n");
>+		buf.append(" }\n");
>+		buf.append(" }\n");
>+		buf.append("}\n");
>+		ICompilationUnit cu= pack1.createCompilationUnit("E.java", buf.toString(), false, null);
>+
>+		int offset= buf.toString().indexOf("!");
>+		int length= "!(a instanceof String)".length();
>+		AssistContext context= getCorrectionContext(cu, offset, length);
>+		List proposals= collectAssists(context, false);
>+
>+		assertCorrectLabels(proposals);
>+
>+		buf= new StringBuffer();
>+		buf.append("package test1;\n");
>+		buf.append("public class E {\n");
>+		buf.append(" public void foo(Object a) {\n");
>+		buf.append(" while (a instanceof String) {\n");
>+		buf.append(" }\n");
>+		buf.append(" }\n");
>+		buf.append("}\n");
>+		String expected1= buf.toString();
>+
>+		assertExpectedExistInProposals(proposals, new String[] { expected1 });
>+
>+	}
>+
>+	public void testInverseCondition4() throws Exception {
>+		IPackageFragment pack1= fSourceFolder.createPackageFragment("test1", false, null);
>+		StringBuffer buf= new StringBuffer();
>+		buf.append("package test1;\n");
>+		buf.append("public class E {\n");
>+		buf.append(" public void foo(Object a) {\n");
>+		buf.append(" for (int i = 0; !(a instanceof String); i++) {\n");
>+		buf.append(" }\n");
>+		buf.append(" }\n");
>+		buf.append("}\n");
>+		ICompilationUnit cu= pack1.createCompilationUnit("E.java", buf.toString(), false, null);
>+
>+		int offset= buf.toString().indexOf("!");
>+		int length= "!(a instanceof String)".length();
>+		AssistContext context= getCorrectionContext(cu, offset, length);
>+		List proposals= collectAssists(context, false);
>+
>+		assertCorrectLabels(proposals);
>+
>+		buf= new StringBuffer();
>+		buf.append("package test1;\n");
>+		buf.append("public class E {\n");
>+		buf.append(" public void foo(Object a) {\n");
>+		buf.append(" for (int i = 0; a instanceof String; i++) {\n");
>+		buf.append(" }\n");
>+		buf.append(" }\n");
>+		buf.append("}\n");
>+		String expected1= buf.toString();
>+
>+		assertExpectedExistInProposals(proposals, new String[] { expected1 });
>+
>+	}
>+
>+	public void testInverseCondition5() throws Exception {
>+		IPackageFragment pack1= fSourceFolder.createPackageFragment("test1", false, null);
>+		StringBuffer buf= new StringBuffer();
>+		buf.append("package test1;\n");
>+		buf.append("public class E {\n");
>+		buf.append(" public void foo(Object a) {\n");
>+		buf.append(" do {\n");
>+		buf.append(" } while (!(a instanceof String));\n");
>+		buf.append(" }\n");
>+		buf.append("}\n");
>+		ICompilationUnit cu= pack1.createCompilationUnit("E.java", buf.toString(), false, null);
>+
>+		int offset= buf.toString().indexOf("!");
>+		int length= "!(a instanceof String)".length();
>+		AssistContext context= getCorrectionContext(cu, offset, length);
>+		List proposals= collectAssists(context, false);
>+
>+		assertCorrectLabels(proposals);
>+
>+		buf= new StringBuffer();
>+		buf.append("package test1;\n");
>+		buf.append("public class E {\n");
>+		buf.append(" public void foo(Object a) {\n");
>+		buf.append(" do {\n");
>+		buf.append(" } while (a instanceof String);\n");
>+		buf.append(" }\n");
>+		buf.append("}\n");
>+		String expected1= buf.toString();
>+
>+		assertExpectedExistInProposals(proposals, new String[] { expected1 });
>+
>+	}
>+
>+	public void testInverseCondition6() throws Exception {
>+		IPackageFragment pack1= fSourceFolder.createPackageFragment("test1", false, null);
>+		StringBuffer buf= new StringBuffer();
>+		buf.append("package test1;\n");
>+		buf.append("public class E {\n");
>+		buf.append(" public void foo(Object a) {\n");
>+		buf.append(" assert !(a instanceof String);\n");
>+		buf.append(" }\n");
>+		buf.append("}\n");
>+		ICompilationUnit cu= pack1.createCompilationUnit("E.java", buf.toString(), false, null);
>+
>+		int offset= buf.toString().indexOf("!");
>+		int length= "!(a instanceof String)".length();
>+		AssistContext context= getCorrectionContext(cu, offset, length);
>+		List proposals= collectAssists(context, false);
>+
>+		assertCorrectLabels(proposals);
>+
>+		buf= new StringBuffer();
>+		buf.append("package test1;\n");
>+		buf.append("public class E {\n");
>+		buf.append(" public void foo(Object a) {\n");
>+		buf.append(" assert a instanceof String;\n");
>+		buf.append(" }\n");
>+		buf.append("}\n");
>+		String expected1= buf.toString();
>+
>+		assertExpectedExistInProposals(proposals, new String[] { expected1 });
>+
>+	}
>+
> 	public void testPushNegationDown1() throws Exception {
> 		IPackageFragment pack1= fSourceFolder.createPackageFragment("test1", false, null);
> 		StringBuffer buf= new StringBuffer();
>Index: ui/org/eclipse/jdt/ui/tests/quickfix/CleanUpTest.java
>===================================================================
>RCS file: /cvsroot/eclipse/org.eclipse.jdt.ui.tests/ui/org/eclipse/jdt/ui/tests/quickfix/CleanUpTest.java,v
>retrieving revision 1.137
>diff -u -r1.137 CleanUpTest.java
>--- ui/org/eclipse/jdt/ui/tests/quickfix/CleanUpTest.java	29 Jan 2011 06:23:03 -0000	1.137
>+++ ui/org/eclipse/jdt/ui/tests/quickfix/CleanUpTest.java	21 Feb 2011 04:10:13 -0000
>@@ -877,6 +877,91 @@
> 		assertRefactoringResultAsExpected(new ICompilationUnit[] { cu1 }, new String[] { expected1 });
> 	}
> 
>+	public void testUnusedCodeBug335173_1() throws Exception {
>+		IPackageFragment pack1= fSourceFolder.createPackageFragment("test1", false, null);
>+		StringBuffer buf= new StringBuffer();
>+
>+		buf.append("package test1;\n");
>+		buf.append("import java.util.Comparator;\n");
>+		buf.append("\n");
>+		buf.append("class IntComp implements Comparator<Integer> {\n");
>+		buf.append(" public int compare(Integer o1, Integer o2) {\n");
>+		buf.append(" return ((Integer) o1).intValue() - ((Integer) o2).intValue();\n");
>+		buf.append(" }\n");
>+		buf.append("}\n");
>+
>+		ICompilationUnit cu1= pack1.createCompilationUnit("IntComp.java", buf.toString(), false, null);
>+
>+		enable(CleanUpConstants.REMOVE_UNNECESSARY_CASTS);
>+
>+		buf= new StringBuffer();
>+		buf.append("package test1;\n");
>+		buf.append("import java.util.Comparator;\n");
>+		buf.append("\n");
>+		buf.append("class IntComp implements Comparator<Integer> {\n");
>+		buf.append(" public int compare(Integer o1, Integer o2) {\n");
>+		buf.append(" return o1.intValue() - o2.intValue();\n");
>+		buf.append(" }\n");
>+		buf.append("}\n");
>+		String expected1= buf.toString();
>+
>+		assertRefactoringResultAsExpected(new ICompilationUnit[] { cu1 }, new String[] { expected1 });
>+	}
>+
>+	public void testUnusedCodeBug335173_2() throws Exception {
>+		IPackageFragment pack1= fSourceFolder.createPackageFragment("test1", false, null);
>+		StringBuffer buf= new StringBuffer();
>+		buf.append("package test1;\n");
>+		buf.append("\n");
>+		buf.append("public class E1 {\n");
>+		buf.append(" public void foo(Integer n) {\n");
>+		buf.append(" int i = (((Integer) n)).intValue();\n");
>+		buf.append(" }\n");
>+		buf.append("}\n");
>+		ICompilationUnit cu1= pack1.createCompilationUnit("E1.java", buf.toString(), false, null);
>+
>+		enable(CleanUpConstants.REMOVE_UNNECESSARY_CASTS);
>+
>+		buf= new StringBuffer();
>+		buf.append("package test1;\n");
>+		buf.append("\n");
>+		buf.append("public class E1 {\n");
>+		buf.append(" public void foo(Integer n) {\n");
>+		buf.append(" int i = (n).intValue();\n");
>+		buf.append(" }\n");
>+		buf.append("}\n");
>+		String expected1= buf.toString();
>+
>+		assertRefactoringResultAsExpected(new ICompilationUnit[] { cu1 }, new String[] { expected1 });
>+	}
>+
>+	public void testUnusedCodeBug335173_3() throws Exception {
>+		IPackageFragment pack1= fSourceFolder.createPackageFragment("test1", false, null);
>+		StringBuffer buf= new StringBuffer();
>+		buf.append("package test1;\n");
>+		buf.append("\n");
>+		buf.append("public class E1 {\n");
>+		buf.append(" public void foo(Integer n) {\n");
>+		buf.append(" int i = ((Integer) (n)).intValue();\n");
>+		buf.append(" }\n");
>+		buf.append("}\n");
>+		ICompilationUnit cu1= pack1.createCompilationUnit("E1.java", buf.toString(), false, null);
>+
>+		enable(CleanUpConstants.REMOVE_UNNECESSARY_CASTS);
>+
>+		buf= new StringBuffer();
>+		buf.append("package test1;\n");
>+		buf.append("\n");
>+		buf.append("public class E1 {\n");
>+		buf.append(" public void foo(Integer n) {\n");
>+		buf.append(" int i = (n).intValue();\n");
>+		buf.append(" }\n");
>+		buf.append("}\n");
>+		String expected1= buf.toString();
>+
>+		assertRefactoringResultAsExpected(new ICompilationUnit[] { cu1 }, new String[] { expected1 });
>+	}
>+
> 	public void testJava5001() throws Exception {
> 		IPackageFragment pack1= fSourceFolder.createPackageFragment("test1", false, null);
> 		StringBuffer buf= new StringBuffer();
>@@ -5868,7 +5953,7 @@
> 		buf.append("package test1;\n");
> 		buf.append("public class E1 {\n");
> 		buf.append(" public void foo() {\n");
>-		buf.append(" double d = 2.0 * 0.5 / 4.0;\n");
>+		buf.append(" double d = 2.0 * (0.5 / 4.0);\n");
> 		buf.append(" int spaceCount = 3;\n");
> 		buf.append(" spaceCount = 2 * (spaceCount / 2);\n");
> 		buf.append(" }\n");
>@@ -5937,6 +6022,624 @@
> 		assertRefactoringResultAsExpected(new ICompilationUnit[] { cu1 }, new String[] { buf.toString() });
> 	}
> 
>+	public void testRemoveParenthesesBug335173_1() throws Exception {
>+		//while loop's expression
>+		IPackageFragment pack1= fSourceFolder.createPackageFragment("test", false, null);
>+		StringBuffer buf= new StringBuffer();
>+		buf.append("package test;\n");
>+		buf.append("public class E {\n");
>+		buf.append(" public void foo(boolean a) {\n");
>+		buf.append(" while (((a))) {\n");
>+		buf.append(" }\n");
>+		buf.append(" }\n");
>+		buf.append(" public void bar(int x) {\n");
>+		buf.append(" while ((x > 2)) {\n");
>+		buf.append(" }\n");
>+		buf.append(" }\n");
>+		buf.append("}\n");
>+		ICompilationUnit cu1= pack1.createCompilationUnit("E.java", buf.toString(), false, null);
>+
>+		enable(CleanUpConstants.EXPRESSIONS_USE_PARENTHESES);
>+		enable(CleanUpConstants.EXPRESSIONS_USE_PARENTHESES_NEVER);
>+
>+		buf= new StringBuffer();
>+		buf.append("package test;\n");
>+		buf.append("public class E {\n");
>+		buf.append(" public void foo(boolean a) {\n");
>+		buf.append(" while (a) {\n");
>+		buf.append(" }\n");
>+		buf.append(" }\n");
>+		buf.append(" public void bar(int x) {\n");
>+		buf.append(" while (x > 2) {\n");
>+		buf.append(" }\n");
>+		buf.append(" }\n");
>+		buf.append("}\n");
>+		String expected1= buf.toString();
>+
>+		assertRefactoringResultAsExpected(new ICompilationUnit[] { cu1 }, new String[] { expected1 });
>+	}
>+
>+	public void testRemoveParenthesesBug335173_2() throws Exception {
>+		//do while loop's expression
>+		IPackageFragment pack1= fSourceFolder.createPackageFragment("test", false, null);
>+		StringBuffer buf= new StringBuffer();
>+		buf.append("package test;\n");
>+		buf.append("public class E {\n");
>+		buf.append(" public void foo(int x) {\n");
>+		buf.append(" do {\n");
>+		buf.append(" } while ((x > 2));\n");
>+		buf.append(" }\n");
>+		buf.append("}\n");
>+		ICompilationUnit cu1= pack1.createCompilationUnit("E.java", buf.toString(), false, null);
>+
>+		enable(CleanUpConstants.EXPRESSIONS_USE_PARENTHESES);
>+		enable(CleanUpConstants.EXPRESSIONS_USE_PARENTHESES_NEVER);
>+
>+		buf= new StringBuffer();
>+		buf.append("package test;\n");
>+		buf.append("public class E {\n");
>+		buf.append(" public void foo(int x) {\n");
>+		buf.append(" do {\n");
>+		buf.append(" } while (x > 2);\n");
>+		buf.append(" }\n");
>+		buf.append("}\n");
>+		String expected1= buf.toString();
>+
>+		assertRefactoringResultAsExpected(new ICompilationUnit[] { cu1 }, new String[] { expected1 });
>+	}
>+
>+	public void testRemoveParenthesesBug335173_3() throws Exception {
>+		//for loop's expression
>+		IPackageFragment pack1= fSourceFolder.createPackageFragment("test", false, null);
>+		StringBuffer buf= new StringBuffer();
>+		buf.append("package test;\n");
>+		buf.append("public class E {\n");
>+		buf.append(" public void foo(int x) {\n");
>+		buf.append(" for (int x = 0; (x > 2); x++) {\n");
>+		buf.append(" }\n");
>+		buf.append(" }\n");
>+		buf.append("}\n");
>+		ICompilationUnit cu1= pack1.createCompilationUnit("E.java", buf.toString(), false, null);
>+
>+		enable(CleanUpConstants.EXPRESSIONS_USE_PARENTHESES);
>+		enable(CleanUpConstants.EXPRESSIONS_USE_PARENTHESES_NEVER);
>+
>+		buf= new StringBuffer();
>+		buf.append("package test;\n");
>+		buf.append("public class E {\n");
>+		buf.append(" public void foo(int x) {\n");
>+		buf.append(" for (int x = 0; x > 2; x++) {\n");
>+		buf.append(" }\n");
>+		buf.append(" }\n");
>+		buf.append("}\n");
>+		String expected1= buf.toString();
>+
>+		assertRefactoringResultAsExpected(new ICompilationUnit[] { cu1 }, new String[] { expected1 });
>+	}
>+
>+	public void testRemoveParenthesesBug335173_4() throws Exception {
>+		//switch statement expression
>+		IPackageFragment pack1= fSourceFolder.createPackageFragment("test", false, null);
>+		StringBuffer buf= new StringBuffer();
>+		buf.append("package test;\n");
>+		buf.append("public class E {\n");
>+		buf.append(" public void foo(int x) {\n");
>+		buf.append(" switch ((x - 2)) {\n");
>+		buf.append(" }\n");
>+		buf.append(" }\n");
>+		buf.append("}\n");
>+		ICompilationUnit cu1= pack1.createCompilationUnit("E.java", buf.toString(), false, null);
>+
>+		enable(CleanUpConstants.EXPRESSIONS_USE_PARENTHESES);
>+		enable(CleanUpConstants.EXPRESSIONS_USE_PARENTHESES_NEVER);
>+
>+		buf= new StringBuffer();
>+		buf.append("package test;\n");
>+		buf.append("public class E {\n");
>+		buf.append(" public void foo(int x) {\n");
>+		buf.append(" switch (x - 2) {\n");
>+		buf.append(" }\n");
>+		buf.append(" }\n");
>+		buf.append("}\n");
>+		String expected1= buf.toString();
>+
>+		assertRefactoringResultAsExpected(new ICompilationUnit[] { cu1 }, new String[] { expected1 });
>+	}
>+
>+	public void testRemoveParenthesesBug335173_5() throws Exception {
>+		//switch case expression
>+		IPackageFragment pack1= fSourceFolder.createPackageFragment("test", false, null);
>+		StringBuffer buf= new StringBuffer();
>+		buf.append("package test;\n");
>+		buf.append("public class E {\n");
>+		buf.append(" public void foo(int x) {\n");
>+		buf.append(" switch (x) {\n");
>+		buf.append(" case (1 + 2):\n");
>+		buf.append(" break;\n");
>+		buf.append(" }\n");
>+		buf.append(" }\n");
>+		buf.append("}\n");
>+		ICompilationUnit cu1= pack1.createCompilationUnit("E.java", buf.toString(), false, null);
>+
>+		enable(CleanUpConstants.EXPRESSIONS_USE_PARENTHESES);
>+		enable(CleanUpConstants.EXPRESSIONS_USE_PARENTHESES_NEVER);
>+
>+		buf= new StringBuffer();
>+		buf.append("package test;\n");
>+		buf.append("public class E {\n");
>+		buf.append(" public void foo(int x) {\n");
>+		buf.append(" switch (x) {\n");
>+		buf.append(" case 1 + 2:\n");
>+		buf.append(" break;\n");
>+		buf.append(" }\n");
>+		buf.append(" }\n");
>+		buf.append("}\n");
>+		String expected1= buf.toString();
>+
>+		assertRefactoringResultAsExpected(new ICompilationUnit[] { cu1 }, new String[] { expected1 });
>+	}
>+
>+	public void testRemoveParenthesesBug335173_6() throws Exception {
>+		//throw statement expression
>+		IPackageFragment pack1= fSourceFolder.createPackageFragment("test", false, null);
>+		StringBuffer buf= new StringBuffer();
>+		buf.append("package test;\n");
>+		buf.append("public class E {\n");
>+		buf.append(" public void foo(int type) throws Exception {\n");
>+		buf.append(" throw (type == 1 ? new IllegalArgumentException() : new Exception());\n");
>+		buf.append(" }\n");
>+		buf.append("}\n");
>+		ICompilationUnit cu1= pack1.createCompilationUnit("E.java", buf.toString(), false, null);
>+
>+		enable(CleanUpConstants.EXPRESSIONS_USE_PARENTHESES);
>+		enable(CleanUpConstants.EXPRESSIONS_USE_PARENTHESES_NEVER);
>+
>+		buf= new StringBuffer();
>+		buf.append("package test;\n");
>+		buf.append("public class E {\n");
>+		buf.append(" public void foo(int type) throws Exception {\n");
>+		buf.append(" throw type == 1 ? new IllegalArgumentException() : new Exception();\n");
>+		buf.append(" }\n");
>+		buf.append("}\n");
>+		String expected1= buf.toString();
>+
>+		assertRefactoringResultAsExpected(new ICompilationUnit[] { cu1 }, new String[] { expected1 });
>+	}
>+
>+	public void testRemoveParenthesesBug335173_7() throws Exception {
>+		//synchronized statement expression
>+		IPackageFragment pack1= fSourceFolder.createPackageFragment("test", false, null);
>+		StringBuffer buf= new StringBuffer();
>+		buf.append("package test;\n");
>+		buf.append("public class E {\n");
>+		buf.append(" private static final Object OBJECT = new Object();\n");
>+		buf.append(" private static final String STRING = new String();\n");
>+		buf.append(" \n");
>+		buf.append(" public void foo(int x) {\n");
>+		buf.append(" synchronized ((x == 1 ? STRING : OBJECT)) {\n");
>+		buf.append(" }\n");
>+		buf.append(" }\n");
>+		buf.append("}\n");
>+		ICompilationUnit cu1= pack1.createCompilationUnit("E.java", buf.toString(), false, null);
>+
>+		enable(CleanUpConstants.EXPRESSIONS_USE_PARENTHESES);
>+		enable(CleanUpConstants.EXPRESSIONS_USE_PARENTHESES_NEVER);
>+
>+		buf= new StringBuffer();
>+		buf.append("package test;\n");
>+		buf.append("public class E {\n");
>+		buf.append(" private static final Object OBJECT = new Object();\n");
>+		buf.append(" private static final String STRING = new String();\n");
>+		buf.append(" \n");
>+		buf.append(" public void foo(int x) {\n");
>+		buf.append(" synchronized (x == 1 ? STRING : OBJECT) {\n");
>+		buf.append(" }\n");
>+		buf.append(" }\n");
>+		buf.append("}\n");
>+		String expected1= buf.toString();
>+
>+		assertRefactoringResultAsExpected(new ICompilationUnit[] { cu1 }, new String[] { expected1 });
>+	}
>+
>+	public void testRemoveParenthesesBug335173_8() throws Exception {
>+		//assert statement expression
>+		IPackageFragment pack1= fSourceFolder.createPackageFragment("test", false, null);
>+		StringBuffer buf= new StringBuffer();
>+		buf.append("package test;\n");
>+		buf.append("public class E {\n");
>+		buf.append(" public void foo(int x) {\n");
>+		buf.append(" assert (x > 2);\n");
>+		buf.append(" }\n");
>+		buf.append("}\n");
>+		ICompilationUnit cu1= pack1.createCompilationUnit("E.java", buf.toString(), false, null);
>+
>+		enable(CleanUpConstants.EXPRESSIONS_USE_PARENTHESES);
>+		enable(CleanUpConstants.EXPRESSIONS_USE_PARENTHESES_NEVER);
>+
>+		buf= new StringBuffer();
>+		buf.append("package test;\n");
>+		buf.append("public class E {\n");
>+		buf.append(" public void foo(int x) {\n");
>+		buf.append(" assert x > 2;\n");
>+		buf.append(" }\n");
>+		buf.append("}\n");
>+		String expected1= buf.toString();
>+
>+		assertRefactoringResultAsExpected(new ICompilationUnit[] { cu1 }, new String[] { expected1 });
>+	}
>+
>+	public void testRemoveParenthesesBug335173_9() throws Exception {
>+		//assert statement message expression
>+		IPackageFragment pack1= fSourceFolder.createPackageFragment("test", false, null);
>+		StringBuffer buf= new StringBuffer();
>+		buf.append("package test;\n");
>+		buf.append("public class E {\n");
>+		buf.append(" public void foo(int x) {\n");
>+		buf.append(" assert x > 2 : (x - 2);\n");
>+		buf.append(" }\n");
>+		buf.append("}\n");
>+		ICompilationUnit cu1= pack1.createCompilationUnit("E.java", buf.toString(), false, null);
>+
>+		enable(CleanUpConstants.EXPRESSIONS_USE_PARENTHESES);
>+		enable(CleanUpConstants.EXPRESSIONS_USE_PARENTHESES_NEVER);
>+
>+		buf= new StringBuffer();
>+		buf.append("package test;\n");
>+		buf.append("public class E {\n");
>+		buf.append(" public void foo(int x) {\n");
>+		buf.append(" assert x > 2 : x - 2;\n");
>+		buf.append(" }\n");
>+		buf.append("}\n");
>+		String expected1= buf.toString();
>+
>+		assertRefactoringResultAsExpected(new ICompilationUnit[] { cu1 }, new String[] { expected1 });
>+	}
>+
>+	public void testRemoveParenthesesBug335173_10() throws Exception {
>+		//array access index expression
>+		IPackageFragment pack1= fSourceFolder.createPackageFragment("test", false, null);
>+		StringBuffer buf= new StringBuffer();
>+		buf.append("package test;\n");
>+		buf.append("public class E {\n");
>+		buf.append(" public void foo(int a[], int x) {\n");
>+		buf.append(" int i = a[(x + 2)];\n");
>+		buf.append(" }\n");
>+		buf.append("}\n");
>+		ICompilationUnit cu1= pack1.createCompilationUnit("E.java", buf.toString(), false, null);
>+
>+		enable(CleanUpConstants.EXPRESSIONS_USE_PARENTHESES);
>+		enable(CleanUpConstants.EXPRESSIONS_USE_PARENTHESES_NEVER);
>+
>+		buf= new StringBuffer();
>+		buf.append("package test;\n");
>+		buf.append("public class E {\n");
>+		buf.append(" public void foo(int a[], int x) {\n");
>+		buf.append(" int i = a[x + 2];\n");
>+		buf.append(" }\n");
>+		buf.append("}\n");
>+		String expected1= buf.toString();
>+
>+		assertRefactoringResultAsExpected(new ICompilationUnit[] { cu1 }, new String[] { expected1 });
>+	}
>+
>+	public void testRemoveParenthesesBug335173_11() throws Exception {
>+		//conditional expression's then expression
>+		IPackageFragment pack1= fSourceFolder.createPackageFragment("test", false, null);
>+		StringBuffer buf= new StringBuffer();
>+		buf.append("package test;\n");
>+		buf.append("public class E {\n");
>+		buf.append(" public void foo(int x) {\n");
>+		buf.append(" int i = x > 10 ? (x > 5 ? x - 1 : x - 2): x;\n");
>+		buf.append(" }\n");
>+		buf.append("}\n");
>+		ICompilationUnit cu1= pack1.createCompilationUnit("E.java", buf.toString(), false, null);
>+
>+		enable(CleanUpConstants.EXPRESSIONS_USE_PARENTHESES);
>+		enable(CleanUpConstants.EXPRESSIONS_USE_PARENTHESES_NEVER);
>+
>+		buf= new StringBuffer();
>+		buf.append("package test;\n");
>+		buf.append("public class E {\n");
>+		buf.append(" public void foo(int x) {\n");
>+		buf.append(" int i = x > 10 ? x > 5 ? x - 1 : x - 2: x;\n");
>+		buf.append(" }\n");
>+		buf.append("}\n");
>+		String expected1= buf.toString();
>+
>+		assertRefactoringResultAsExpected(new ICompilationUnit[] { cu1 }, new String[] { expected1 });
>+	}
>+
>+	public void testRemoveParenthesesBug335173_12() throws Exception {
>+		//conditional expression's else expression
>+		IPackageFragment pack1= fSourceFolder.createPackageFragment("test", false, null);
>+		StringBuffer buf= new StringBuffer();
>+		buf.append("package test;\n");
>+		buf.append("public class E {\n");
>+		buf.append(" public void foo(int x) {\n");
>+		buf.append(" int i = x > 10 ? x: (x > 5 ? x - 1 : x - 2);\n");
>+		buf.append(" }\n");
>+		buf.append("}\n");
>+		ICompilationUnit cu1= pack1.createCompilationUnit("E.java", buf.toString(), false, null);
>+
>+		enable(CleanUpConstants.EXPRESSIONS_USE_PARENTHESES);
>+		enable(CleanUpConstants.EXPRESSIONS_USE_PARENTHESES_NEVER);
>+
>+		buf= new StringBuffer();
>+		buf.append("package test;\n");
>+		buf.append("public class E {\n");
>+		buf.append(" public void foo(int x) {\n");
>+		buf.append(" int i = x > 10 ? x: x > 5 ? x - 1 : x - 2;\n");
>+		buf.append(" }\n");
>+		buf.append("}\n");
>+		String expected1= buf.toString();
>+
>+		assertRefactoringResultAsExpected(new ICompilationUnit[] { cu1 }, new String[] { expected1 });
>+	}
>+
>+	public void testRemoveParenthesesBug335173_13() throws Exception {
>+		//conditional expression's then expression
>+		IPackageFragment pack1= fSourceFolder.createPackageFragment("test", false, null);
>+		StringBuffer buf= new StringBuffer();
>+		buf.append("package test;\n");
>+		buf.append("public class E {\n");
>+		buf.append(" public void foo(int x) {\n");
>+		buf.append(" int i = x > 10 ? (x = x - 2): x;\n");
>+		buf.append(" }\n");
>+		buf.append("}\n");
>+		ICompilationUnit cu1= pack1.createCompilationUnit("E.java", buf.toString(), false, null);
>+
>+		enable(CleanUpConstants.EXPRESSIONS_USE_PARENTHESES);
>+		enable(CleanUpConstants.EXPRESSIONS_USE_PARENTHESES_NEVER);
>+
>+		buf= new StringBuffer();
>+		buf.append("package test;\n");
>+		buf.append("public class E {\n");
>+		buf.append(" public void foo(int x) {\n");
>+		buf.append(" int i = x > 10 ? (x = x - 2): x;\n");
>+		buf.append(" }\n");
>+		buf.append("}\n");
>+		String expected1= buf.toString();
>+
>+		assertRefactoringResultAsExpected(new ICompilationUnit[] { cu1 }, new String[] { expected1 });
>+	}
>+
>+	public void testRemoveParenthesesBug335173_14() throws Exception {
>+		//conditional expression's else expression
>+		IPackageFragment pack1= fSourceFolder.createPackageFragment("test", false, null);
>+		StringBuffer buf= new StringBuffer();
>+		buf.append("package test;\n");
>+		buf.append("public class E {\n");
>+		buf.append(" public void foo(int x) {\n");
>+		buf.append(" int i = x > 10 ? x: (x = x - 2);\n");
>+		buf.append(" }\n");
>+		buf.append("}\n");
>+		ICompilationUnit cu1= pack1.createCompilationUnit("E.java", buf.toString(), false, null);
>+
>+		enable(CleanUpConstants.EXPRESSIONS_USE_PARENTHESES);
>+		enable(CleanUpConstants.EXPRESSIONS_USE_PARENTHESES_NEVER);
>+
>+		buf= new StringBuffer();
>+		buf.append("package test;\n");
>+		buf.append("public class E {\n");
>+		buf.append(" public void foo(int x) {\n");
>+		buf.append(" int i = x > 10 ? x: (x = x - 2);\n");
>+		buf.append(" }\n");
>+		buf.append("}\n");
>+		String expected1= buf.toString();
>+
>+		assertRefactoringResultAsExpected(new ICompilationUnit[] { cu1 }, new String[] { expected1 });
>+	}
>+
>+	public void testRemoveParenthesesBug335173_15() throws Exception {
>+		//shift operators
>+		IPackageFragment pack1= fSourceFolder.createPackageFragment("test", false, null);
>+		StringBuffer buf= new StringBuffer();
>+		buf.append("package test;\n");
>+		buf.append("public class E {\n");
>+		buf.append(" public void foo(int x) {\n");
>+		buf.append(" int m= (x >> 2) >> 1;\n");
>+		buf.append(" m= x >> (2 >> 1);\n");
>+		buf.append(" int n= (x << 2) << 1;\n");
>+		buf.append(" n= x << (2 << 1);\n");
>+		buf.append(" }\n");
>+		buf.append("}\n");
>+		ICompilationUnit cu1= pack1.createCompilationUnit("E.java", buf.toString(), false, null);
>+
>+		enable(CleanUpConstants.EXPRESSIONS_USE_PARENTHESES);
>+		enable(CleanUpConstants.EXPRESSIONS_USE_PARENTHESES_NEVER);
>+
>+		buf= new StringBuffer();
>+		buf.append("package test;\n");
>+		buf.append("public class E {\n");
>+		buf.append(" public void foo(int x) {\n");
>+		buf.append(" int m= x >> 2 >> 1;\n");
>+		buf.append(" m= x >> (2 >> 1);\n");
>+		buf.append(" int n= x << 2 << 1;\n");
>+		buf.append(" n= x << (2 << 1);\n");
>+		buf.append(" }\n");
>+		buf.append("}\n");
>+		String expected1= buf.toString();
>+
>+		assertRefactoringResultAsExpected(new ICompilationUnit[] { cu1 }, new String[] { expected1 });
>+	}
>+
>+	public void testRemoveParenthesesBug335173_16() throws Exception {
>+		//integer multiplication
>+		IPackageFragment pack1= fSourceFolder.createPackageFragment("test", false, null);
>+		StringBuffer buf= new StringBuffer();
>+		buf.append("package test;\n");
>+		buf.append("public class E {\n");
>+		buf.append(" public void foo(int x, long y) {\n");
>+		buf.append(" int m= (4 * x) * 2;\n");
>+		buf.append(" int n= 4 * (x * 2);\n");
>+		buf.append(" int p= 4 * (x % 3);\n");
>+		buf.append(" int q= 4 * (x / 3);\n");
>+		buf.append(" int r= 4 * (x * y);\n");
>+		buf.append(" }\n");
>+		buf.append("}\n");
>+		ICompilationUnit cu1= pack1.createCompilationUnit("E.java", buf.toString(), false, null);
>+
>+		enable(CleanUpConstants.EXPRESSIONS_USE_PARENTHESES);
>+		enable(CleanUpConstants.EXPRESSIONS_USE_PARENTHESES_NEVER);
>+
>+		buf= new StringBuffer();
>+		buf.append("package test;\n");
>+		buf.append("public class E {\n");
>+		buf.append(" public void foo(int x, long y) {\n");
>+		buf.append(" int m= 4 * x * 2;\n");
>+		buf.append(" int n= 4 * x * 2;\n");
>+		buf.append(" int p= 4 * (x % 3);\n");
>+		buf.append(" int q= 4 * (x / 3);\n");
>+		buf.append(" int r= 4 * (x * y);\n");
>+		buf.append(" }\n");
>+		buf.append("}\n");
>+		String expected1= buf.toString();
>+
>+		assertRefactoringResultAsExpected(new ICompilationUnit[] { cu1 }, new String[] { expected1 });
>+	}
>+
>+	public void testRemoveParenthesesBug335173_17() throws Exception {
>+		//floating point multiplication
>+		IPackageFragment pack1= fSourceFolder.createPackageFragment("test", false, null);
>+		StringBuffer buf= new StringBuffer();
>+		buf.append("package test;\n");
>+		buf.append("public class E {\n");
>+		buf.append(" public void foo(double x) {\n");
>+		buf.append(" int m= (4.0 * x) * 0.5;\n");
>+		buf.append(" int n= 4.0 * (x * 0.5);\n");
>+		buf.append(" int p= 4.0 * (x / 100);\n");
>+		buf.append(" int q= 4.0 * (x % 3);\n");
>+		buf.append(" }\n");
>+		buf.append("}\n");
>+		ICompilationUnit cu1= pack1.createCompilationUnit("E.java", buf.toString(), false, null);
>+
>+		enable(CleanUpConstants.EXPRESSIONS_USE_PARENTHESES);
>+		enable(CleanUpConstants.EXPRESSIONS_USE_PARENTHESES_NEVER);
>+
>+		buf= new StringBuffer();
>+		buf.append("package test;\n");
>+		buf.append("public class E {\n");
>+		buf.append(" public void foo(double x) {\n");
>+		buf.append(" int m= 4.0 * x * 0.5;\n");
>+		buf.append(" int n= 4.0 * (x * 0.5);\n");
>+		buf.append(" int p= 4.0 * (x / 100);\n");
>+		buf.append(" int q= 4.0 * (x % 3);\n");
>+		buf.append(" }\n");
>+		buf.append("}\n");
>+		String expected1= buf.toString();
>+
>+		assertRefactoringResultAsExpected(new ICompilationUnit[] { cu1 }, new String[] { expected1 });
>+	}
>+
>+	public void testRemoveParenthesesBug335173_18() throws Exception {
>+		//integer addition
>+		IPackageFragment pack1= fSourceFolder.createPackageFragment("test", false, null);
>+		StringBuffer buf= new StringBuffer();
>+		buf.append("package test;\n");
>+		buf.append("public class E {\n");
>+		buf.append(" public void foo(int x, long y) {\n");
>+		buf.append(" int m= (4 + x) + 2;\n");
>+		buf.append(" int n= 4 + (x + 2);\n");
>+		buf.append(" int p= 4 + (x + y);\n");
>+		buf.append(" }\n");
>+		buf.append("}\n");
>+		ICompilationUnit cu1= pack1.createCompilationUnit("E.java", buf.toString(), false, null);
>+
>+		enable(CleanUpConstants.EXPRESSIONS_USE_PARENTHESES);
>+		enable(CleanUpConstants.EXPRESSIONS_USE_PARENTHESES_NEVER);
>+
>+		buf= new StringBuffer();
>+		buf.append("package test;\n");
>+		buf.append("public class E {\n");
>+		buf.append(" public void foo(int x, long y) {\n");
>+		buf.append(" int m= 4 + x + 2;\n");
>+		buf.append(" int n= 4 + x + 2;\n");
>+		buf.append(" int p= 4 + (x + y);\n");
>+		buf.append(" }\n");
>+		buf.append("}\n");
>+		String expected1= buf.toString();
>+
>+		assertRefactoringResultAsExpected(new ICompilationUnit[] { cu1 }, new String[] { expected1 });
>+	}
>+
>+	public void testRemoveParenthesesBug335173_19() throws Exception {
>+		//floating point addition
>+		IPackageFragment pack1= fSourceFolder.createPackageFragment("test", false, null);
>+		StringBuffer buf= new StringBuffer();
>+		buf.append("package test;\n");
>+		buf.append("public class E {\n");
>+		buf.append(" public void foo(double x) {\n");
>+		buf.append(" int m= (4.0 + x) + 100.0;\n");
>+		buf.append(" int n= 4.0 + (x + 100.0);\n");
>+		buf.append(" }\n");
>+		buf.append("}\n");
>+		ICompilationUnit cu1= pack1.createCompilationUnit("E.java", buf.toString(), false, null);
>+
>+		enable(CleanUpConstants.EXPRESSIONS_USE_PARENTHESES);
>+		enable(CleanUpConstants.EXPRESSIONS_USE_PARENTHESES_NEVER);
>+
>+		buf= new StringBuffer();
>+		buf.append("package test;\n");
>+		buf.append("public class E {\n");
>+		buf.append(" public void foo(double x) {\n");
>+		buf.append(" int m= 4.0 + x + 100.0;\n");
>+		buf.append(" int n= 4.0 + (x + 100.0);\n");
>+		buf.append(" }\n");
>+		buf.append("}\n");
>+		String expected1= buf.toString();
>+
>+		assertRefactoringResultAsExpected(new ICompilationUnit[] { cu1 }, new String[] { expected1 });
>+	}
>+
>+	public void testRemoveParenthesesBug335173_20() throws Exception {
>+		//string concatenation
>+		IPackageFragment pack1= fSourceFolder.createPackageFragment("test", false, null);
>+		StringBuffer buf= new StringBuffer();
>+		buf.append("package test;\n");
>+		buf.append("public class E {\n");
>+		buf.append(" public void foo(String s, String t, String u) {\n");
>+		buf.append(" String a= (s + t) + u;\n");
>+		buf.append(" String b= s + (t + u);\n");
>+		buf.append(" String c= (1 + 2) + s;\n");
>+		buf.append(" String d= 1 + (2 + s);\n");
>+		buf.append(" String e= s + (1 + 2);\n");
>+		buf.append(" String f= (s + 1) + 2;\n");
>+		buf.append(" String g= (1 + s) + 2;\n");
>+		buf.append(" String h= 1 + (s + 2);\n");
>+		buf.append(" String i= s + (1 + t);\n");
>+		buf.append(" String j= s + (t + 1);\n");
>+		buf.append(" String k= s + (1 - 2);\n");
>+		buf.append(" String l= s + (1 * 2);\n");
>+		buf.append(" }\n");
>+		buf.append("}\n");
>+		ICompilationUnit cu1= pack1.createCompilationUnit("E.java", buf.toString(), false, null);
>+
>+		enable(CleanUpConstants.EXPRESSIONS_USE_PARENTHESES);
>+		enable(CleanUpConstants.EXPRESSIONS_USE_PARENTHESES_NEVER);
>+
>+		buf= new StringBuffer();
>+		buf.append("package test;\n");
>+		buf.append("public class E {\n");
>+		buf.append(" public void foo(String s, String t, String u) {\n");
>+		buf.append(" String a= s + t + u;\n");
>+		buf.append(" String b= s + t + u;\n");
>+		buf.append(" String c= 1 + 2 + s;\n");
>+		buf.append(" String d= 1 + (2 + s);\n");
>+		buf.append(" String e= s + (1 + 2);\n");
>+		buf.append(" String f= s + 1 + 2;\n");
>+		buf.append(" String g= 1 + s + 2;\n");
>+		buf.append(" String h= 1 + s + 2;\n");
>+		buf.append(" String i= s + 1 + t;\n");
>+		buf.append(" String j= s + t + 1;\n");
>+		buf.append(" String k= s + (1 - 2);\n");
>+		buf.append(" String l= s + 1 * 2;\n");
>+		buf.append(" }\n");
>+		buf.append("}\n");
>+		String expected1= buf.toString();
>+
>+		assertRefactoringResultAsExpected(new ICompilationUnit[] { cu1 }, new String[] { expected1 });
>+	}
>+
> 	public void testRemoveQualifier01() throws Exception {
> 		IPackageFragment pack1= fSourceFolder.createPackageFragment("test1", false, null);
> 		StringBuffer buf= new StringBuffer();
>Index: ui/org/eclipse/jdt/ui/tests/quickfix/GetterSetterQuickFixTest.java
>===================================================================
>RCS file: /cvsroot/eclipse/org.eclipse.jdt.ui.tests/ui/org/eclipse/jdt/ui/tests/quickfix/GetterSetterQuickFixTest.java,v
>retrieving revision 1.4
>diff -u -r1.4 GetterSetterQuickFixTest.java
>--- ui/org/eclipse/jdt/ui/tests/quickfix/GetterSetterQuickFixTest.java	17 Jun 2010 12:56:28 -0000	1.4
>+++ ui/org/eclipse/jdt/ui/tests/quickfix/GetterSetterQuickFixTest.java	21 Feb 2011 04:10:13 -0000
>@@ -1,5 +1,5 @@
> /*******************************************************************************
>- * Copyright (c) 2007, 2010 IBM Corporation and others.
>+ * Copyright (c) 2007, 2011 IBM Corporation and others.
> * All rights reserved. This program and the accompanying materials
> * are made available under the terms of the Eclipse Public License v1.0
> * which accompanies this distribution, and is available at
>@@ -165,6 +165,238 @@
> 		assertExpectedExistInProposals(proposals, expected);
> 	}
> 
>+	public void testInvisibleFieldToGetterSetterBug335173_1() throws Exception {
>+		IPackageFragment pack1= fSourceFolder.createPackageFragment("p", false, null);
>+		StringBuffer buf= new StringBuffer();
>+		buf.append("package p;\n");
>+		buf.append("\n");
>+		buf.append("public class C {\n");
>+		buf.append(" private int test;\n");
>+		buf.append("\n");
>+		buf.append(" public int getTest() {\n");
>+		buf.append(" return this.test;\n");
>+		buf.append(" }\n");
>+		buf.append("\n");
>+		buf.append(" public void setTest(int test) {\n");
>+		buf.append(" this.test = test;\n");
>+		buf.append(" }\n");
>+		buf.append("}\n");
>+		buf.append("\n");
>+		buf.append("class D {\n");
>+		buf.append(" public void foo(int x){\n");
>+		buf.append(" C c=new C();\n");
>+		buf.append(" c.test+= x;\n");
>+		buf.append(" }\n");
>+		buf.append("}\n");
>+		ICompilationUnit cu= pack1.createCompilationUnit("C.java", buf.toString(), false, null);
>+
>+		CompilationUnit astRoot= getASTRoot(cu);
>+		ArrayList proposals= collectCorrections(cu, astRoot);
>+
>+		assertCorrectLabels(proposals);
>+		assertNumberOfProposals(proposals, 2);
>+
>+		String[] expected= new String[1];
>+		buf= new StringBuffer();
>+		buf.append("package p;\n");
>+		buf.append("\n");
>+		buf.append("public class C {\n");
>+		buf.append(" private int test;\n");
>+		buf.append("\n");
>+		buf.append(" public int getTest() {\n");
>+		buf.append(" return this.test;\n");
>+		buf.append(" }\n");
>+		buf.append("\n");
>+		buf.append(" public void setTest(int test) {\n");
>+		buf.append(" this.test = test;\n");
>+		buf.append(" }\n");
>+		buf.append("}\n");
>+		buf.append("\n");
>+		buf.append("class D {\n");
>+		buf.append(" public void foo(int x){\n");
>+		buf.append(" C c=new C();\n");
>+		buf.append(" c.setTest(c.getTest() + x);\n");
>+		buf.append(" }\n");
>+		buf.append("}\n");
>+		expected[0]= buf.toString();
>+
>+		assertExpectedExistInProposals(proposals, expected);
>+	}
>+
>+	public void testInvisibleFieldToGetterSetterBug335173_2() throws Exception {
>+		IPackageFragment pack1= fSourceFolder.createPackageFragment("p", false, null);
>+		StringBuffer buf= new StringBuffer();
>+		buf.append("package p;\n");
>+		buf.append("\n");
>+		buf.append("public class C {\n");
>+		buf.append(" private int test;\n");
>+		buf.append("\n");
>+		buf.append(" public int getTest() {\n");
>+		buf.append(" return this.test;\n");
>+		buf.append(" }\n");
>+		buf.append("\n");
>+		buf.append(" public void setTest(int test) {\n");
>+		buf.append(" this.test = test;\n");
>+		buf.append(" }\n");
>+		buf.append("}\n");
>+		buf.append("\n");
>+		buf.append("class D {\n");
>+		buf.append(" public void foo(){\n");
>+		buf.append(" C c=new C();\n");
>+		buf.append(" c.test+= 1 + 2;\n");
>+		buf.append(" }\n");
>+		buf.append("}\n");
>+		ICompilationUnit cu= pack1.createCompilationUnit("C.java", buf.toString(), false, null);
>+
>+		CompilationUnit astRoot= getASTRoot(cu);
>+		ArrayList proposals= collectCorrections(cu, astRoot);
>+
>+		assertCorrectLabels(proposals);
>+		assertNumberOfProposals(proposals, 2);
>+
>+		String[] expected= new String[1];
>+		buf= new StringBuffer();
>+		buf.append("package p;\n");
>+		buf.append("\n");
>+		buf.append("public class C {\n");
>+		buf.append(" private int test;\n");
>+		buf.append("\n");
>+		buf.append(" public int getTest() {\n");
>+		buf.append(" return this.test;\n");
>+		buf.append(" }\n");
>+		buf.append("\n");
>+		buf.append(" public void setTest(int test) {\n");
>+		buf.append(" this.test = test;\n");
>+		buf.append(" }\n");
>+		buf.append("}\n");
>+		buf.append("\n");
>+		buf.append("class D {\n");
>+		buf.append(" public void foo(){\n");
>+		buf.append(" C c=new C();\n");
>+		buf.append(" c.setTest(c.getTest() + (1 + 2));\n");
>+		buf.append(" }\n");
>+		buf.append("}\n");
>+		expected[0]= buf.toString();
>+
>+		assertExpectedExistInProposals(proposals, expected);
>+	}
>+
>+	public void testInvisibleFieldToGetterSetterBug335173_3() throws Exception {
>+		IPackageFragment pack1= fSourceFolder.createPackageFragment("p", false, null);
>+		StringBuffer buf= new StringBuffer();
>+		buf.append("package p;\n");
>+		buf.append("\n");
>+		buf.append("public class C {\n");
>+		buf.append(" private int test;\n");
>+		buf.append("\n");
>+		buf.append(" public int getTest() {\n");
>+		buf.append(" return this.test;\n");
>+		buf.append(" }\n");
>+		buf.append("\n");
>+		buf.append(" public void setTest(int test) {\n");
>+		buf.append(" this.test = test;\n");
>+		buf.append(" }\n");
>+		buf.append("}\n");
>+		buf.append("\n");
>+		buf.append("class D {\n");
>+		buf.append(" public void foo(){\n");
>+		buf.append(" C c=new C();\n");
>+		buf.append(" c.test-= 1 + 2;\n");
>+		buf.append(" }\n");
>+		buf.append("}\n");
>+		ICompilationUnit cu= pack1.createCompilationUnit("C.java", buf.toString(), false, null);
>+
>+		CompilationUnit astRoot= getASTRoot(cu);
>+		ArrayList proposals= collectCorrections(cu, astRoot);
>+
>+		assertCorrectLabels(proposals);
>+		assertNumberOfProposals(proposals, 2);
>+
>+		String[] expected= new String[1];
>+		buf= new StringBuffer();
>+		buf.append("package p;\n");
>+		buf.append("\n");
>+		buf.append("public class C {\n");
>+		buf.append(" private int test;\n");
>+		buf.append("\n");
>+		buf.append(" public int getTest() {\n");
>+		buf.append(" return this.test;\n");
>+		buf.append(" }\n");
>+		buf.append("\n");
>+		buf.append(" public void setTest(int test) {\n");
>+		buf.append(" this.test = test;\n");
>+		buf.append(" }\n");
>+		buf.append("}\n");
>+		buf.append("\n");
>+		buf.append("class D {\n");
>+		buf.append(" public void foo(){\n");
>+		buf.append(" C c=new C();\n");
>+		buf.append(" c.setTest(c.getTest() - (1 + 2));\n");
>+		buf.append(" }\n");
>+		buf.append("}\n");
>+		expected[0]= buf.toString();
>+
>+		assertExpectedExistInProposals(proposals, expected);
>+	}
>+
>+	public void testInvisibleFieldToGetterSetterBug335173_4() throws Exception {
>+		IPackageFragment pack1= fSourceFolder.createPackageFragment("p", false, null);
>+		StringBuffer buf= new StringBuffer();
>+		buf.append("package p;\n");
>+		buf.append("\n");
>+		buf.append("public class C {\n");
>+		buf.append(" private int test;\n");
>+		buf.append("\n");
>+		buf.append(" public int getTest() {\n");
>+		buf.append(" return this.test;\n");
>+		buf.append(" }\n");
>+		buf.append("\n");
>+		buf.append(" public void setTest(int test) {\n");
>+		buf.append(" this.test = test;\n");
>+		buf.append(" }\n");
>+		buf.append("}\n");
>+		buf.append("\n");
>+		buf.append("class D {\n");
>+		buf.append(" public void foo(){\n");
>+		buf.append(" C c=new C();\n");
>+		buf.append(" c.test*= 1 + 2;\n");
>+		buf.append(" }\n");
>+		buf.append("}\n");
>+		ICompilationUnit cu= pack1.createCompilationUnit("C.java", buf.toString(), false, null);
>+
>+		CompilationUnit astRoot= getASTRoot(cu);
>+		ArrayList proposals= collectCorrections(cu, astRoot);
>+
>+		assertCorrectLabels(proposals);
>+		assertNumberOfProposals(proposals, 2);
>+
>+		String[] expected= new String[1];
>+		buf= new StringBuffer();
>+		buf.append("package p;\n");
>+		buf.append("\n");
>+		buf.append("public class C {\n");
>+		buf.append(" private int test;\n");
>+		buf.append("\n");
>+		buf.append(" public int getTest() {\n");
>+		buf.append(" return this.test;\n");
>+		buf.append(" }\n");
>+		buf.append("\n");
>+		buf.append(" public void setTest(int test) {\n");
>+		buf.append(" this.test = test;\n");
>+		buf.append(" }\n");
>+		buf.append("}\n");
>+		buf.append("\n");
>+		buf.append("class D {\n");
>+		buf.append(" public void foo(){\n");
>+		buf.append(" C c=new C();\n");
>+		buf.append(" c.setTest(c.getTest() * (1 + 2));\n");
>+		buf.append(" }\n");
>+		buf.append("}\n");
>+		expected[0]= buf.toString();
>+
>+		assertExpectedExistInProposals(proposals, expected);
>+	}
>+
> 	public void testCreateFieldUsingSef() throws Exception {
> 		IPackageFragment pack1= fSourceFolder.createPackageFragment("", false, null);
> 		StringBuffer buf= new StringBuffer();
>Index: ui/org/eclipse/jdt/ui/tests/quickfix/LocalCorrectionsQuickFixTest.java
>===================================================================
>RCS file: /cvsroot/eclipse/org.eclipse.jdt.ui.tests/ui/org/eclipse/jdt/ui/tests/quickfix/LocalCorrectionsQuickFixTest.java,v
>retrieving revision 1.171
>diff -u -r1.171 LocalCorrectionsQuickFixTest.java
>--- ui/org/eclipse/jdt/ui/tests/quickfix/LocalCorrectionsQuickFixTest.java	31 Jan 2011 17:50:45 -0000	1.171
>+++ ui/org/eclipse/jdt/ui/tests/quickfix/LocalCorrectionsQuickFixTest.java	21 Feb 2011 04:10:13 -0000
>@@ -4411,6 +4411,72 @@
> 		assertEqualString(preview, buf.toString());
> 	}
> 
>+	public void testUnnecessaryCastBug335173_1() throws Exception {
>+		Hashtable hashtable= JavaCore.getOptions();
>+		hashtable.put(JavaCore.COMPILER_PB_UNNECESSARY_TYPE_CHECK, JavaCore.ERROR);
>+		JavaCore.setOptions(hashtable);
>+
>+		IPackageFragment pack1= fSourceFolder.createPackageFragment("test1", false, null);
>+		StringBuffer buf= new StringBuffer();
>+		buf.append("package test1;\n");
>+		buf.append("public class E {\n");
>+		buf.append(" public void foo(Integer n) {\n");
>+		buf.append(" int i = (((Integer) n)).intValue();\n");
>+		buf.append(" }\n");
>+		buf.append("}\n");
>+		ICompilationUnit cu= pack1.createCompilationUnit("E.java", buf.toString(), false, null);
>+
>+		CompilationUnit astRoot= getASTRoot(cu);
>+		ArrayList proposals= collectCorrections(cu, astRoot);
>+		assertNumberOfProposals(proposals, 1);
>+		assertCorrectLabels(proposals);
>+
>+		CUCorrectionProposal proposal= (CUCorrectionProposal)proposals.get(0);
>+		String preview= getPreviewContent(proposal);
>+
>+		buf= new StringBuffer();
>+		buf.append("package test1;\n");
>+		buf.append("public class E {\n");
>+		buf.append(" public void foo(Integer n) {\n");
>+		buf.append(" int i = (n).intValue();\n");
>+		buf.append(" }\n");
>+		buf.append("}\n");
>+		assertEqualString(preview, buf.toString());
>+	}
>+
>+	public void testUnnecessaryCastBug335173_2() throws Exception {
>+		Hashtable hashtable= JavaCore.getOptions();
>+		hashtable.put(JavaCore.COMPILER_PB_UNNECESSARY_TYPE_CHECK, JavaCore.ERROR);
>+		JavaCore.setOptions(hashtable);
>+
>+		IPackageFragment pack1= fSourceFolder.createPackageFragment("test1", false, null);
>+		StringBuffer buf= new StringBuffer();
>+		buf.append("package test1;\n");
>+		buf.append("public class E {\n");
>+		buf.append(" public void foo(Integer n) {\n");
>+		buf.append(" int i = ((Integer) (n)).intValue();\n");
>+		buf.append(" }\n");
>+		buf.append("}\n");
>+		ICompilationUnit cu= pack1.createCompilationUnit("E.java", buf.toString(), false, null);
>+
>+		CompilationUnit astRoot= getASTRoot(cu);
>+		ArrayList proposals= collectCorrections(cu, astRoot);
>+		assertNumberOfProposals(proposals, 1);
>+		assertCorrectLabels(proposals);
>+
>+		CUCorrectionProposal proposal= (CUCorrectionProposal)proposals.get(0);
>+		String preview= getPreviewContent(proposal);
>+
>+		buf= new StringBuffer();
>+		buf.append("package test1;\n");
>+		buf.append("public class E {\n");
>+		buf.append(" public void foo(Integer n) {\n");
>+		buf.append(" int i = (n).intValue();\n");
>+		buf.append(" }\n");
>+		buf.append("}\n");
>+		assertEqualString(preview, buf.toString());
>+	}
>+
> 	public void testSuperfluousSemicolon() throws Exception {
> 		Hashtable hashtable= JavaCore.getOptions();
> 		hashtable.put(JavaCore.COMPILER_PB_EMPTY_STATEMENT, JavaCore.ERROR);
>#P org.eclipse.jdt.ui.tests.refactoring
>Index: resources/ExtractConstant/canExtract/A_test39_in.java
>===================================================================
>RCS file: resources/ExtractConstant/canExtract/A_test39_in.java
>diff -N resources/ExtractConstant/canExtract/A_test39_in.java
>--- /dev/null	1 Jan 1970 00:00:00 -0000
>+++ resources/ExtractConstant/canExtract/A_test39_in.java	1 Jan 1970 00:00:00 -0000
>@@ -0,0 +1,7 @@
>+//5, 21 -> 5, 26 AllowLoadtime == false
>+package p;
>+class A {
>+	void f() {
>+		int i= 1 - (2 + 3);
>+	}
>+}
>\ No newline at end of file
>Index: resources/ExtractConstant/canExtract/A_test39_out.java
>===================================================================
>RCS file: resources/ExtractConstant/canExtract/A_test39_out.java
>diff -N resources/ExtractConstant/canExtract/A_test39_out.java
>--- /dev/null	1 Jan 1970 00:00:00 -0000
>+++ resources/ExtractConstant/canExtract/A_test39_out.java	1 Jan 1970 00:00:00 -0000
>@@ -0,0 +1,9 @@
>+//5, 21 -> 5, 26 AllowLoadtime == false
>+package p;
>+class A {
>+	private static final int CONSTANT= 2 + 3;
>+
>+	void f() {
>+		int i= 1 - CONSTANT;
>+	}
>+}
>\ No newline at end of file
>Index: resources/ExtractConstant/canExtract/A_test40_in.java
>===================================================================
>RCS file: resources/ExtractConstant/canExtract/A_test40_in.java
>diff -N resources/ExtractConstant/canExtract/A_test40_in.java
>--- /dev/null	1 Jan 1970 00:00:00 -0000
>+++ resources/ExtractConstant/canExtract/A_test40_in.java	1 Jan 1970 00:00:00 -0000
>@@ -0,0 +1,7 @@
>+//5, 20 -> 5, 27 AllowLoadtime == false
>+package p;
>+class A {
>+	void f() {
>+		int i= 1 - (2 + 3);
>+	}
>+}
>\ No newline at end of file
>Index: resources/ExtractConstant/canExtract/A_test40_out.java
>===================================================================
>RCS file: resources/ExtractConstant/canExtract/A_test40_out.java
>diff -N resources/ExtractConstant/canExtract/A_test40_out.java
>--- /dev/null	1 Jan 1970 00:00:00 -0000
>+++ resources/ExtractConstant/canExtract/A_test40_out.java	1 Jan 1970 00:00:00 -0000
>@@ -0,0 +1,9 @@
>+//5, 20 -> 5, 27 AllowLoadtime == false
>+package p;
>+class A {
>+	private static final int CONSTANT= 2 + 3;
>+
>+	void f() {
>+		int i= 1 - CONSTANT;
>+	}
>+}
>\ No newline at end of file
>Index: resources/ExtractConstant/canExtract/A_test41_in.java
>===================================================================
>RCS file: resources/ExtractConstant/canExtract/A_test41_in.java
>diff -N resources/ExtractConstant/canExtract/A_test41_in.java
>--- /dev/null	1 Jan 1970 00:00:00 -0000
>+++ resources/ExtractConstant/canExtract/A_test41_in.java	1 Jan 1970 00:00:00 -0000
>@@ -0,0 +1,7 @@
>+//5, 22 -> 5, 27 AllowLoadtime == false
>+package p;
>+class A {
>+	void f() {
>+		int i= 1 - ((2 + 3));
>+	}
>+}
>\ No newline at end of file
>Index: resources/ExtractConstant/canExtract/A_test41_out.java
>===================================================================
>RCS file: resources/ExtractConstant/canExtract/A_test41_out.java
>diff -N resources/ExtractConstant/canExtract/A_test41_out.java
>--- /dev/null	1 Jan 1970 00:00:00 -0000
>+++ resources/ExtractConstant/canExtract/A_test41_out.java	1 Jan 1970 00:00:00 -0000
>@@ -0,0 +1,9 @@
>+//5, 22 -> 5, 27 AllowLoadtime == false
>+package p;
>+class A {
>+	private static final int CONSTANT= 2 + 3;
>+
>+	void f() {
>+		int i= 1 - (CONSTANT);
>+	}
>+}
>\ No newline at end of file
>Index: resources/ExtractConstant/canExtract/A_test42_in.java
>===================================================================
>RCS file: resources/ExtractConstant/canExtract/A_test42_in.java
>diff -N resources/ExtractConstant/canExtract/A_test42_in.java
>--- /dev/null	1 Jan 1970 00:00:00 -0000
>+++ resources/ExtractConstant/canExtract/A_test42_in.java	1 Jan 1970 00:00:00 -0000
>@@ -0,0 +1,7 @@
>+//5, 21 -> 5, 28 AllowLoadtime == false
>+package p;
>+class A {
>+	void f() {
>+		int i= 1 - ((2 + 3));
>+	}
>+}
>\ No newline at end of file
>Index: resources/ExtractConstant/canExtract/A_test42_out.java
>===================================================================
>RCS file: resources/ExtractConstant/canExtract/A_test42_out.java
>diff -N resources/ExtractConstant/canExtract/A_test42_out.java
>--- /dev/null	1 Jan 1970 00:00:00 -0000
>+++ resources/ExtractConstant/canExtract/A_test42_out.java	1 Jan 1970 00:00:00 -0000
>@@ -0,0 +1,9 @@
>+//5, 21 -> 5, 28 AllowLoadtime == false
>+package p;
>+class A {
>+	private static final int CONSTANT= 2 + 3;
>+
>+	void f() {
>+		int i= 1 - CONSTANT;
>+	}
>+}
>\ No newline at end of file
>Index: resources/ExtractConstant/canExtract/A_test43_in.java
>===================================================================
>RCS file: resources/ExtractConstant/canExtract/A_test43_in.java
>diff -N resources/ExtractConstant/canExtract/A_test43_in.java
>--- /dev/null	1 Jan 1970 00:00:00 -0000
>+++ resources/ExtractConstant/canExtract/A_test43_in.java	1 Jan 1970 00:00:00 -0000
>@@ -0,0 +1,7 @@
>+//5, 20 -> 5, 29 AllowLoadtime == false
>+package p;
>+class A {
>+	void f() {
>+		int i= 1 - ((2 + 3));
>+	}
>+}
>\ No newline at end of file
>Index: resources/ExtractConstant/canExtract/A_test43_out.java
>===================================================================
>RCS file: resources/ExtractConstant/canExtract/A_test43_out.java
>diff -N resources/ExtractConstant/canExtract/A_test43_out.java
>--- /dev/null	1 Jan 1970 00:00:00 -0000
>+++ resources/ExtractConstant/canExtract/A_test43_out.java	1 Jan 1970 00:00:00 -0000
>@@ -0,0 +1,9 @@
>+//5, 20 -> 5, 29 AllowLoadtime == false
>+package p;
>+class A {
>+	private static final int CONSTANT= (2 + 3);
>+
>+	void f() {
>+		int i= 1 - CONSTANT;
>+	}
>+}
>\ No newline at end of file
>Index: resources/ExtractMethodWorkSpace/ExtractMethodTests/expression_in/A_test625.java
>===================================================================
>RCS file: resources/ExtractMethodWorkSpace/ExtractMethodTests/expression_in/A_test625.java
>diff -N resources/ExtractMethodWorkSpace/ExtractMethodTests/expression_in/A_test625.java
>--- /dev/null	1 Jan 1970 00:00:00 -0000
>+++ resources/ExtractMethodWorkSpace/ExtractMethodTests/expression_in/A_test625.java	1 Jan 1970 00:00:00 -0000
>@@ -0,0 +1,9 @@
>+package expression_in;
>+
>+public class A_test625 {
>+
>+	public void foo() {
>+		int i= 1 - (/*[*/2 + 3/*]*/);
>+		int j= 1 - (2 + 3);
>+	}	
>+}
>Index: resources/ExtractMethodWorkSpace/ExtractMethodTests/expression_in/A_test626.java
>===================================================================
>RCS file: resources/ExtractMethodWorkSpace/ExtractMethodTests/expression_in/A_test626.java
>diff -N resources/ExtractMethodWorkSpace/ExtractMethodTests/expression_in/A_test626.java
>--- /dev/null	1 Jan 1970 00:00:00 -0000
>+++ resources/ExtractMethodWorkSpace/ExtractMethodTests/expression_in/A_test626.java	1 Jan 1970 00:00:00 -0000
>@@ -0,0 +1,9 @@
>+package expression_in;
>+
>+public class A_test626 {
>+
>+	public void foo() {
>+		int i= 1 - /*[*/(2 + 3)/*]*/;
>+		int j= 1 - (2 + 3);
>+	}	
>+}
>Index: resources/ExtractMethodWorkSpace/ExtractMethodTests/expression_in/A_test627.java
>===================================================================
>RCS file: resources/ExtractMethodWorkSpace/ExtractMethodTests/expression_in/A_test627.java
>diff -N resources/ExtractMethodWorkSpace/ExtractMethodTests/expression_in/A_test627.java
>--- /dev/null	1 Jan 1970 00:00:00 -0000
>+++ resources/ExtractMethodWorkSpace/ExtractMethodTests/expression_in/A_test627.java	1 Jan 1970 00:00:00 -0000
>@@ -0,0 +1,9 @@
>+package expression_in;
>+
>+public class A_test627 {
>+
>+	public void foo() {
>+		int i= 1 - ((/*[*/2 + 3/*]*/));
>+		int j= 1 - ((2 + 3));
>+	}	
>+}
>Index: resources/ExtractMethodWorkSpace/ExtractMethodTests/expression_in/A_test628.java
>===================================================================
>RCS file: resources/ExtractMethodWorkSpace/ExtractMethodTests/expression_in/A_test628.java
>diff -N resources/ExtractMethodWorkSpace/ExtractMethodTests/expression_in/A_test628.java
>--- /dev/null	1 Jan 1970 00:00:00 -0000
>+++ resources/ExtractMethodWorkSpace/ExtractMethodTests/expression_in/A_test628.java	1 Jan 1970 00:00:00 -0000
>@@ -0,0 +1,9 @@
>+package expression_in;
>+
>+public class A_test627 {
>+
>+	public void foo() {
>+		int i= 1 - (/*[*/(2 + 3)/*]*/);
>+		int j= 1 - ((2 + 3));
>+	}	
>+}
>Index: resources/ExtractMethodWorkSpace/ExtractMethodTests/expression_in/A_test629.java
>===================================================================
>RCS file: resources/ExtractMethodWorkSpace/ExtractMethodTests/expression_in/A_test629.java
>diff -N resources/ExtractMethodWorkSpace/ExtractMethodTests/expression_in/A_test629.java
>--- /dev/null	1 Jan 1970 00:00:00 -0000
>+++ resources/ExtractMethodWorkSpace/ExtractMethodTests/expression_in/A_test629.java	1 Jan 1970 00:00:00 -0000
>@@ -0,0 +1,9 @@
>+package expression_in;
>+
>+public class A_test627 {
>+
>+	public void foo() {
>+		int i= 1 - /*[*/((2 + 3))/*]*/;
>+		int j= 1 - ((2 + 3));
>+	}	
>+}
>Index: resources/ExtractMethodWorkSpace/ExtractMethodTests/expression_out/A_test625.java
>===================================================================
>RCS file: resources/ExtractMethodWorkSpace/ExtractMethodTests/expression_out/A_test625.java
>diff -N resources/ExtractMethodWorkSpace/ExtractMethodTests/expression_out/A_test625.java
>--- /dev/null	1 Jan 1970 00:00:00 -0000
>+++ resources/ExtractMethodWorkSpace/ExtractMethodTests/expression_out/A_test625.java	1 Jan 1970 00:00:00 -0000
>@@ -0,0 +1,13 @@
>+package expression_out;
>+
>+public class A_test625 {
>+
>+	public void foo() {
>+		int i= 1 - extracted();
>+		int j= 1 - extracted();
>+	}
>+
>+	protected int extracted() {
>+		return /*[*/2 + 3/*]*/;
>+	}	
>+}
>Index: resources/ExtractMethodWorkSpace/ExtractMethodTests/expression_out/A_test626.java
>===================================================================
>RCS file: resources/ExtractMethodWorkSpace/ExtractMethodTests/expression_out/A_test626.java
>diff -N resources/ExtractMethodWorkSpace/ExtractMethodTests/expression_out/A_test626.java
>--- /dev/null	1 Jan 1970 00:00:00 -0000
>+++ resources/ExtractMethodWorkSpace/ExtractMethodTests/expression_out/A_test626.java	1 Jan 1970 00:00:00 -0000
>@@ -0,0 +1,13 @@
>+package expression_out;
>+
>+public class A_test626 {
>+
>+	public void foo() {
>+		int i= 1 - extracted();
>+		int j= 1 - extracted();
>+	}
>+
>+	protected int extracted() {
>+		return 2 + 3;
>+	}	
>+}
>Index: resources/ExtractMethodWorkSpace/ExtractMethodTests/expression_out/A_test627.java
>===================================================================
>RCS file: resources/ExtractMethodWorkSpace/ExtractMethodTests/expression_out/A_test627.java
>diff -N resources/ExtractMethodWorkSpace/ExtractMethodTests/expression_out/A_test627.java
>--- /dev/null	1 Jan 1970 00:00:00 -0000
>+++ resources/ExtractMethodWorkSpace/ExtractMethodTests/expression_out/A_test627.java	1 Jan 1970 00:00:00 -0000
>@@ -0,0 +1,13 @@
>+package expression_out;
>+
>+public class A_test627 {
>+
>+	public void foo() {
>+		int i= 1 - (extracted());
>+		int j= 1 - (extracted());
>+	}
>+
>+	protected int extracted() {
>+		return /*[*/2 + 3/*]*/;
>+	}	
>+}
>Index: resources/ExtractMethodWorkSpace/ExtractMethodTests/expression_out/A_test628.java
>===================================================================
>RCS file: resources/ExtractMethodWorkSpace/ExtractMethodTests/expression_out/A_test628.java
>diff -N resources/ExtractMethodWorkSpace/ExtractMethodTests/expression_out/A_test628.java
>--- /dev/null	1 Jan 1970 00:00:00 -0000
>+++ resources/ExtractMethodWorkSpace/ExtractMethodTests/expression_out/A_test628.java	1 Jan 1970 00:00:00 -0000
>@@ -0,0 +1,13 @@
>+package expression_out;
>+
>+public class A_test627 {
>+
>+	public void foo() {
>+		int i= 1 - extracted();
>+		int j= 1 - extracted();
>+	}
>+
>+	protected int extracted() {
>+		return 2 + 3;
>+	}	
>+}
>Index: resources/ExtractMethodWorkSpace/ExtractMethodTests/expression_out/A_test629.java
>===================================================================
>RCS file: resources/ExtractMethodWorkSpace/ExtractMethodTests/expression_out/A_test629.java
>diff -N resources/ExtractMethodWorkSpace/ExtractMethodTests/expression_out/A_test629.java
>--- /dev/null	1 Jan 1970 00:00:00 -0000
>+++ resources/ExtractMethodWorkSpace/ExtractMethodTests/expression_out/A_test629.java	1 Jan 1970 00:00:00 -0000
>@@ -0,0 +1,13 @@
>+package expression_out;
>+
>+public class A_test627 {
>+
>+	public void foo() {
>+		int i= 1 - extracted();
>+		int j= 1 - extracted();
>+	}
>+
>+	protected int extracted() {
>+		return (2 + 3);
>+	}	
>+}
>Index: resources/ExtractTemp/canExtract/A_test105_in.java
>===================================================================
>RCS file: resources/ExtractTemp/canExtract/A_test105_in.java
>diff -N resources/ExtractTemp/canExtract/A_test105_in.java
>--- /dev/null	1 Jan 1970 00:00:00 -0000
>+++ resources/ExtractTemp/canExtract/A_test105_in.java	1 Jan 1970 00:00:00 -0000
>@@ -0,0 +1,7 @@
>+package p; //5, 21, 5, 26
>+
>+class A {
>+	public void m() {
>+		int a= 1 - (2 + 3);
>+	}
>+}
>Index: resources/ExtractTemp/canExtract/A_test105_out.java
>===================================================================
>RCS file: resources/ExtractTemp/canExtract/A_test105_out.java
>diff -N resources/ExtractTemp/canExtract/A_test105_out.java
>--- /dev/null	1 Jan 1970 00:00:00 -0000
>+++ resources/ExtractTemp/canExtract/A_test105_out.java	1 Jan 1970 00:00:00 -0000
>@@ -0,0 +1,8 @@
>+package p; //5, 21, 5, 26
>+
>+class A {
>+	public void m() {
>+		int temp= 2 + 3;
>+		int a= 1 - temp;
>+	}
>+}
>Index: resources/ExtractTemp/canExtract/A_test106_in.java
>===================================================================
>RCS file: resources/ExtractTemp/canExtract/A_test106_in.java
>diff -N resources/ExtractTemp/canExtract/A_test106_in.java
>--- /dev/null	1 Jan 1970 00:00:00 -0000
>+++ resources/ExtractTemp/canExtract/A_test106_in.java	1 Jan 1970 00:00:00 -0000
>@@ -0,0 +1,7 @@
>+package p; //5, 20, 5, 27
>+
>+class A {
>+	public void m() {
>+		int a= 1 - (2 + 3);
>+	}
>+}
>Index: resources/ExtractTemp/canExtract/A_test106_out.java
>===================================================================
>RCS file: resources/ExtractTemp/canExtract/A_test106_out.java
>diff -N resources/ExtractTemp/canExtract/A_test106_out.java
>--- /dev/null	1 Jan 1970 00:00:00 -0000
>+++ resources/ExtractTemp/canExtract/A_test106_out.java	1 Jan 1970 00:00:00 -0000
>@@ -0,0 +1,8 @@
>+package p; //5, 20, 5, 27
>+
>+class A {
>+	public void m() {
>+		int temp= 2 + 3;
>+		int a= 1 - temp;
>+	}
>+}
>Index: resources/ExtractTemp/canExtract/A_test107_in.java
>===================================================================
>RCS file: resources/ExtractTemp/canExtract/A_test107_in.java
>diff -N resources/ExtractTemp/canExtract/A_test107_in.java
>--- /dev/null	1 Jan 1970 00:00:00 -0000
>+++ resources/ExtractTemp/canExtract/A_test107_in.java	1 Jan 1970 00:00:00 -0000
>@@ -0,0 +1,7 @@
>+package p; //5, 22, 5, 27
>+
>+class A {
>+	public void m() {
>+		int a= 1 - ((2 + 3));
>+	}
>+}
>Index: resources/ExtractTemp/canExtract/A_test107_out.java
>===================================================================
>RCS file: resources/ExtractTemp/canExtract/A_test107_out.java
>diff -N resources/ExtractTemp/canExtract/A_test107_out.java
>--- /dev/null	1 Jan 1970 00:00:00 -0000
>+++ resources/ExtractTemp/canExtract/A_test107_out.java	1 Jan 1970 00:00:00 -0000
>@@ -0,0 +1,8 @@
>+package p; //5, 22, 5, 27
>+
>+class A {
>+	public void m() {
>+		int temp= 2 + 3;
>+		int a= 1 - (temp);
>+	}
>+}
>Index: resources/ExtractTemp/canExtract/A_test108_in.java
>===================================================================
>RCS file: resources/ExtractTemp/canExtract/A_test108_in.java
>diff -N resources/ExtractTemp/canExtract/A_test108_in.java
>--- /dev/null	1 Jan 1970 00:00:00 -0000
>+++ resources/ExtractTemp/canExtract/A_test108_in.java	1 Jan 1970 00:00:00 -0000
>@@ -0,0 +1,7 @@
>+package p; //5, 21, 5, 28
>+
>+class A {
>+	public void m() {
>+		int a= 1 - ((2 + 3));
>+	}
>+}
>Index: resources/ExtractTemp/canExtract/A_test108_out.java
>===================================================================
>RCS file: resources/ExtractTemp/canExtract/A_test108_out.java
>diff -N resources/ExtractTemp/canExtract/A_test108_out.java
>--- /dev/null	1 Jan 1970 00:00:00 -0000
>+++ resources/ExtractTemp/canExtract/A_test108_out.java	1 Jan 1970 00:00:00 -0000
>@@ -0,0 +1,8 @@
>+package p; //5, 21, 5, 28
>+
>+class A {
>+	public void m() {
>+		int temp= 2 + 3;
>+		int a= 1 - temp;
>+	}
>+}
>Index: resources/ExtractTemp/canExtract/A_test109_in.java
>===================================================================
>RCS file: resources/ExtractTemp/canExtract/A_test109_in.java
>diff -N resources/ExtractTemp/canExtract/A_test109_in.java
>--- /dev/null	1 Jan 1970 00:00:00 -0000
>+++ resources/ExtractTemp/canExtract/A_test109_in.java	1 Jan 1970 00:00:00 -0000
>@@ -0,0 +1,7 @@
>+package p; //5, 20, 5, 29
>+
>+class A {
>+	public void m() {
>+		int a= 1 - ((2 + 3));
>+	}
>+}
>Index: resources/ExtractTemp/canExtract/A_test109_out.java
>===================================================================
>RCS file: resources/ExtractTemp/canExtract/A_test109_out.java
>diff -N resources/ExtractTemp/canExtract/A_test109_out.java
>--- /dev/null	1 Jan 1970 00:00:00 -0000
>+++ resources/ExtractTemp/canExtract/A_test109_out.java	1 Jan 1970 00:00:00 -0000
>@@ -0,0 +1,8 @@
>+package p; //5, 20, 5, 29
>+
>+class A {
>+	public void m() {
>+		int temp= (2 + 3);
>+		int a= 1 - temp;
>+	}
>+}
>Index: resources/InlineConstant/canInline/test24/out/A.java
>===================================================================
>RCS file: /cvsroot/eclipse/org.eclipse.jdt.ui.tests.refactoring/resources/InlineConstant/canInline/test24/out/A.java,v
>retrieving revision 1.3
>diff -u -r1.3 A.java
>--- resources/InlineConstant/canInline/test24/out/A.java	25 Feb 2005 17:32:45 -0000	1.3
>+++ resources/InlineConstant/canInline/test24/out/A.java	21 Feb 2011 04:10:15 -0000
>@@ -14,7 +14,7 @@
> 				+ p.A.getCount() + A.getCount() + getCount2();
> 		int ii= Consts.I + Consts.I + q.Consts.I
> 				+ p.A.getCount() + A.getCount() + getCount2();
>-		return (I + Consts.I + q.Consts.I
>-				+ p.A.getCount() + A.getCount() + getCount2()) + i + ii;
>+		return I + Consts.I + q.Consts.I
>+				+ p.A.getCount() + A.getCount() + getCount2() + i + ii;
> 	}
> }
>Index: resources/InlineConstant/canInline/test33/out/A.java
>===================================================================
>RCS file: /cvsroot/eclipse/org.eclipse.jdt.ui.tests.refactoring/resources/InlineConstant/canInline/test33/out/A.java,v
>retrieving revision 1.1
>diff -u -r1.1 A.java
>--- resources/InlineConstant/canInline/test33/out/A.java	16 Jun 2009 17:11:37 -0000	1.1
>+++ resources/InlineConstant/canInline/test33/out/A.java	21 Feb 2011 04:10:15 -0000
>@@ -8,10 +8,10 @@
> int f1= K - 1 - (B - 1);
> int f2= K - (B - 1) - (B - 1) - (B - 1);
> 
>- int x1= K + (B - 1);
>+ int x1= K + B - 1;
> int x2= K - (B - 1);
> int x3= K + 1 - (B - 1);
>- int x4= K - 1 + (B - 1);
>- int x5= K + 1 + (B - 1) - (B - 1) - (B - 1);
>+ int x4= K - 1 + B - 1;
>+ int x5= K + 1 + B - 1 - (B - 1) - (B - 1);
> }
> }
>Index: resources/InlineConstant/canInline/test35/in/A.java
>===================================================================
>RCS file: resources/InlineConstant/canInline/test35/in/A.java
>diff -N resources/InlineConstant/canInline/test35/in/A.java
>--- /dev/null	1 Jan 1970 00:00:00 -0000
>+++ resources/InlineConstant/canInline/test35/in/A.java	1 Jan 1970 00:00:00 -0000
>@@ -0,0 +1,8 @@
>+package p;
>+
>+class A {
>+	static final int a = 10 + 20;
>+	void foo() {
>+		int b = 1 + a;
>+	}
>+}
>\ No newline at end of file
>Index: resources/InlineConstant/canInline/test35/out/A.java
>===================================================================
>RCS file: resources/InlineConstant/canInline/test35/out/A.java
>diff -N resources/InlineConstant/canInline/test35/out/A.java
>--- /dev/null	1 Jan 1970 00:00:00 -0000
>+++ resources/InlineConstant/canInline/test35/out/A.java	1 Jan 1970 00:00:00 -0000
>@@ -0,0 +1,7 @@
>+package p;
>+
>+class A {
>+	void foo() {
>+		int b = 1 + 10 + 20;
>+	}
>+}
>\ No newline at end of file
>Index: resources/InlineConstant/canInline/test36/in/A.java
>===================================================================
>RCS file: resources/InlineConstant/canInline/test36/in/A.java
>diff -N resources/InlineConstant/canInline/test36/in/A.java
>--- /dev/null	1 Jan 1970 00:00:00 -0000
>+++ resources/InlineConstant/canInline/test36/in/A.java	1 Jan 1970 00:00:00 -0000
>@@ -0,0 +1,8 @@
>+package p;
>+
>+class A {
>+	static final int a = 10 + 20;
>+	void foo() {
>+		int b = 1 - a;
>+	}
>+}
>\ No newline at end of file
>Index: resources/InlineConstant/canInline/test36/out/A.java
>===================================================================
>RCS file: resources/InlineConstant/canInline/test36/out/A.java
>diff -N resources/InlineConstant/canInline/test36/out/A.java
>--- /dev/null	1 Jan 1970 00:00:00 -0000
>+++ resources/InlineConstant/canInline/test36/out/A.java	1 Jan 1970 00:00:00 -0000
>@@ -0,0 +1,7 @@
>+package p;
>+
>+class A {
>+	void foo() {
>+		int b = 1 - (10 + 20);
>+	}
>+}
>\ No newline at end of file
>Index: resources/InlineConstant/canInline/test37/in/A.java
>===================================================================
>RCS file: resources/InlineConstant/canInline/test37/in/A.java
>diff -N resources/InlineConstant/canInline/test37/in/A.java
>--- /dev/null	1 Jan 1970 00:00:00 -0000
>+++ resources/InlineConstant/canInline/test37/in/A.java	1 Jan 1970 00:00:00 -0000
>@@ -0,0 +1,7 @@
>+package p;
>+
>+class A {
>+	static final long CONST = 2 * Integer.MAX_VALUE;
>+	
>+	long much = CONST * Integer.MAX_VALUE;
>+}
>Index: resources/InlineConstant/canInline/test37/out/A.java
>===================================================================
>RCS file: resources/InlineConstant/canInline/test37/out/A.java
>diff -N resources/InlineConstant/canInline/test37/out/A.java
>--- /dev/null	1 Jan 1970 00:00:00 -0000
>+++ resources/InlineConstant/canInline/test37/out/A.java	1 Jan 1970 00:00:00 -0000
>@@ -0,0 +1,5 @@
>+package p;
>+
>+class A {
>+	long much = (long) (2 * Integer.MAX_VALUE) * Integer.MAX_VALUE;
>+}
>Index: resources/InlineMethodWorkspace/TestCases/argument_out/TestArray.java
>===================================================================
>RCS file: /cvsroot/eclipse/org.eclipse.jdt.ui.tests.refactoring/resources/InlineMethodWorkspace/TestCases/argument_out/TestArray.java,v
>retrieving revision 1.3
>diff -u -r1.3 TestArray.java
>--- resources/InlineMethodWorkspace/TestCases/argument_out/TestArray.java	25 Feb 2005 17:32:43 -0000	1.3
>+++ resources/InlineMethodWorkspace/TestCases/argument_out/TestArray.java	21 Feb 2011 04:10:16 -0000
>@@ -7,6 +7,6 @@
> 	}
> 	
> 	public void main() {
>-		int i= (new int[] {1})[0];
>+		int i= new int[] {1}[0];
> 	}
> }
>Index: resources/InlineMethodWorkspace/TestCases/cast_in/TestInfixExpression1.java
>===================================================================
>RCS file: resources/InlineMethodWorkspace/TestCases/cast_in/TestInfixExpression1.java
>diff -N resources/InlineMethodWorkspace/TestCases/cast_in/TestInfixExpression1.java
>--- /dev/null	1 Jan 1970 00:00:00 -0000
>+++ resources/InlineMethodWorkspace/TestCases/cast_in/TestInfixExpression1.java	1 Jan 1970 00:00:00 -0000
>@@ -0,0 +1,11 @@
>+package cast_in;
>+
>+public class TestInfixExpression1 {	
>+	long x(long two) {
>+		return two * Integer.MAX_VALUE;
>+	}
>+
>+	void foo() {
>+		long much = /*]*/x(1 + 1)/*[*/;
>+	}
>+}
>Index: resources/InlineMethodWorkspace/TestCases/cast_in/TestInfixExpression2.java
>===================================================================
>RCS file: resources/InlineMethodWorkspace/TestCases/cast_in/TestInfixExpression2.java
>diff -N resources/InlineMethodWorkspace/TestCases/cast_in/TestInfixExpression2.java
>--- /dev/null	1 Jan 1970 00:00:00 -0000
>+++ resources/InlineMethodWorkspace/TestCases/cast_in/TestInfixExpression2.java	1 Jan 1970 00:00:00 -0000
>@@ -0,0 +1,11 @@
>+package cast_in;
>+
>+public class TestInfixExpression2 {	
>+	long x(long two) {
>+		return two * Integer.MAX_VALUE;
>+	}
>+
>+	void foo() {
>+		long much = /*]*/x(2 * Integer.MAX_VALUE)/*[*/;
>+	}
>+}
>Index: resources/InlineMethodWorkspace/TestCases/cast_out/TestInfixExpression1.java
>===================================================================
>RCS file: resources/InlineMethodWorkspace/TestCases/cast_out/TestInfixExpression1.java
>diff -N resources/InlineMethodWorkspace/TestCases/cast_out/TestInfixExpression1.java
>--- /dev/null	1 Jan 1970 00:00:00 -0000
>+++ resources/InlineMethodWorkspace/TestCases/cast_out/TestInfixExpression1.java	1 Jan 1970 00:00:00 -0000
>@@ -0,0 +1,11 @@
>+package cast_out;
>+
>+public class TestInfixExpression1 {	
>+	long x(long two) {
>+		return two * Integer.MAX_VALUE;
>+	}
>+
>+	void foo() {
>+		long much = (long) (1 + 1) * Integer.MAX_VALUE;
>+	}
>+}
>Index: resources/InlineMethodWorkspace/TestCases/cast_out/TestInfixExpression2.java
>===================================================================
>RCS file: resources/InlineMethodWorkspace/TestCases/cast_out/TestInfixExpression2.java
>diff -N resources/InlineMethodWorkspace/TestCases/cast_out/TestInfixExpression2.java
>--- /dev/null	1 Jan 1970 00:00:00 -0000
>+++ resources/InlineMethodWorkspace/TestCases/cast_out/TestInfixExpression2.java	1 Jan 1970 00:00:00 -0000
>@@ -0,0 +1,11 @@
>+package cast_out;
>+
>+public class TestInfixExpression2 {	
>+	long x(long two) {
>+		return two * Integer.MAX_VALUE;
>+	}
>+
>+	void foo() {
>+		long much = (long) (2 * Integer.MAX_VALUE) * Integer.MAX_VALUE;
>+	}
>+}
>Index: resources/InlineMethodWorkspace/TestCases/expression_out/TestConditionalExpression.java
>===================================================================
>RCS file: /cvsroot/eclipse/org.eclipse.jdt.ui.tests.refactoring/resources/InlineMethodWorkspace/TestCases/expression_out/TestConditionalExpression.java,v
>retrieving revision 1.3
>diff -u -r1.3 TestConditionalExpression.java
>--- resources/InlineMethodWorkspace/TestCases/expression_out/TestConditionalExpression.java	25 Feb 2005 17:32:45 -0000	1.3
>+++ resources/InlineMethodWorkspace/TestCases/expression_out/TestConditionalExpression.java	21 Feb 2011 04:10:18 -0000
>@@ -5,6 +5,6 @@
> 		return k == 3 ? s.hashCode() : 3;
> 	}
> 	void f(int p) {
>-		int u = (p == 3 ? this.hashCode() : 3);
>+		int u = p == 3 ? this.hashCode() : 3;
> 	}
> }
>\ No newline at end of file
>Index: resources/InlineMethodWorkspace/TestCases/operator_in/TestDiffPlus.java
>===================================================================
>RCS file: resources/InlineMethodWorkspace/TestCases/operator_in/TestDiffPlus.java
>diff -N resources/InlineMethodWorkspace/TestCases/operator_in/TestDiffPlus.java
>--- /dev/null	1 Jan 1970 00:00:00 -0000
>+++ resources/InlineMethodWorkspace/TestCases/operator_in/TestDiffPlus.java	1 Jan 1970 00:00:00 -0000
>@@ -0,0 +1,13 @@
>+package operator_in;
>+
>+public class TestPlusPlus {
>+	int result;
>+	
>+	public void foo() {
>+		result= /*]*/inline(20 - 10)/*[*/;
>+	}
>+	
>+	public int inline(int x) {
>+		return 1 + x;
>+	}
>+}
>\ No newline at end of file
>Index: resources/InlineMethodWorkspace/TestCases/operator_in/TestPlusPlus_1.java
>===================================================================
>RCS file: resources/InlineMethodWorkspace/TestCases/operator_in/TestPlusPlus_1.java
>diff -N resources/InlineMethodWorkspace/TestCases/operator_in/TestPlusPlus_1.java
>--- /dev/null	1 Jan 1970 00:00:00 -0000
>+++ resources/InlineMethodWorkspace/TestCases/operator_in/TestPlusPlus_1.java	1 Jan 1970 00:00:00 -0000
>@@ -0,0 +1,13 @@
>+package operator_in;
>+
>+public class TestPlusPlus_1 {
>+	double result;
>+	
>+	public void foo() {
>+		result= /*]*/inline(10.1 + 10.1)/*[*/;
>+	}
>+	
>+	public double inline(double x) {
>+		return 1.1 + x;
>+	}
>+}
>\ No newline at end of file
>Index: resources/InlineMethodWorkspace/TestCases/operator_out/TestDiffPlus.java
>===================================================================
>RCS file: resources/InlineMethodWorkspace/TestCases/operator_out/TestDiffPlus.java
>diff -N resources/InlineMethodWorkspace/TestCases/operator_out/TestDiffPlus.java
>--- /dev/null	1 Jan 1970 00:00:00 -0000
>+++ resources/InlineMethodWorkspace/TestCases/operator_out/TestDiffPlus.java	1 Jan 1970 00:00:00 -0000
>@@ -0,0 +1,13 @@
>+package operator_out;
>+
>+public class TestPlusPlus {
>+	int result;
>+	
>+	public void foo() {
>+		result= /*]*/1 + 20 - 10/*[*/;
>+	}
>+	
>+	public int inline(int x) {
>+		return 1 + x;
>+	}
>+}
>\ No newline at end of file
>Index: resources/InlineMethodWorkspace/TestCases/operator_out/TestPlusPlus.java
>===================================================================
>RCS file: /cvsroot/eclipse/org.eclipse.jdt.ui.tests.refactoring/resources/InlineMethodWorkspace/TestCases/operator_out/TestPlusPlus.java,v
>retrieving revision 1.2
>diff -u -r1.2 TestPlusPlus.java
>--- resources/InlineMethodWorkspace/TestCases/operator_out/TestPlusPlus.java	17 Nov 2010 05:26:26 -0000	1.2
>+++ resources/InlineMethodWorkspace/TestCases/operator_out/TestPlusPlus.java	21 Feb 2011 04:10:19 -0000
>@@ -4,7 +4,7 @@
> 	int result;
> 	
> 	public void foo() {
>-		result= /*]*/1 + (10 + 10)/*[*/;
>+		result= /*]*/1 + 10 + 10/*[*/;
> 	}
> 	
> 	public int inline(int x) {
>Index: resources/InlineMethodWorkspace/TestCases/operator_out/TestPlusPlus_1.java
>===================================================================
>RCS file: resources/InlineMethodWorkspace/TestCases/operator_out/TestPlusPlus_1.java
>diff -N resources/InlineMethodWorkspace/TestCases/operator_out/TestPlusPlus_1.java
>--- /dev/null	1 Jan 1970 00:00:00 -0000
>+++ resources/InlineMethodWorkspace/TestCases/operator_out/TestPlusPlus_1.java	1 Jan 1970 00:00:00 -0000
>@@ -0,0 +1,13 @@
>+package operator_out;
>+
>+public class TestPlusPlus_1 {
>+	double result;
>+	
>+	public void foo() {
>+		result= /*]*/1.1 + (10.1 + 10.1)/*[*/;
>+	}
>+	
>+	public double inline(double x) {
>+		return 1.1 + x;
>+	}
>+}
>\ No newline at end of file
>Index: resources/InlineMethodWorkspace/TestCases/operator_out/TestTimesTimes.java
>===================================================================
>RCS file: /cvsroot/eclipse/org.eclipse.jdt.ui.tests.refactoring/resources/InlineMethodWorkspace/TestCases/operator_out/TestTimesTimes.java,v
>retrieving revision 1.1
>diff -u -r1.1 TestTimesTimes.java
>--- resources/InlineMethodWorkspace/TestCases/operator_out/TestTimesTimes.java	17 Nov 2010 05:26:26 -0000	1.1
>+++ resources/InlineMethodWorkspace/TestCases/operator_out/TestTimesTimes.java	21 Feb 2011 04:10:20 -0000
>@@ -4,7 +4,7 @@
> 	int result;
> 	
> 	public void foo() {
>-		result= /*]*/1 * (10 * 10)/*[*/;
>+		result= /*]*/1 * 10 * 10/*[*/;
> 	}
> 	
> 	public int inline(int x) {
>Index: resources/InlineMethodWorkspace/TestCases/simple_out/TestComment2.java
>===================================================================
>RCS file: /cvsroot/eclipse/org.eclipse.jdt.ui.tests.refactoring/resources/InlineMethodWorkspace/TestCases/simple_out/TestComment2.java,v
>retrieving revision 1.1
>diff -u -r1.1 TestComment2.java
>--- resources/InlineMethodWorkspace/TestCases/simple_out/TestComment2.java	21 Jul 2010 06:02:38 -0000	1.1
>+++ resources/InlineMethodWorkspace/TestCases/simple_out/TestComment2.java	21 Feb 2011 04:10:20 -0000
>@@ -2,6 +2,6 @@
> 
> public class TestComment2 {
> 	public void ref() {
>-		int toInline = 42 * (5 /*op1*/ * /*op2*/ 2);
>+		int toInline = 42 * 5 /*op1*/ * /*op2*/ 2;
> 	}
> }
>Index: resources/InlineTemp/canInline/A_test36_in.java
>===================================================================
>RCS file: /cvsroot/eclipse/org.eclipse.jdt.ui.tests.refactoring/resources/InlineTemp/canInline/A_test36_in.java,v
>retrieving revision 1.1
>diff -u -r1.1 A_test36_in.java
>--- resources/InlineTemp/canInline/A_test36_in.java	23 Feb 2009 13:32:17 -0000	1.1
>+++ resources/InlineTemp/canInline/A_test36_in.java	21 Feb 2011 04:10:20 -0000
>@@ -1,7 +1,5 @@
> package p;
> 
>-import java.util.List;
>-
> class A {
> 	void x() {
> 		long two = 1 + 1;
>Index: resources/InlineTemp/canInline/A_test36_out.java
>===================================================================
>RCS file: /cvsroot/eclipse/org.eclipse.jdt.ui.tests.refactoring/resources/InlineTemp/canInline/A_test36_out.java,v
>retrieving revision 1.1
>diff -u -r1.1 A_test36_out.java
>--- resources/InlineTemp/canInline/A_test36_out.java	23 Feb 2009 13:32:17 -0000	1.1
>+++ resources/InlineTemp/canInline/A_test36_out.java	21 Feb 2011 04:10:20 -0000
>@@ -1,7 +1,5 @@
> package p;
> 
>-import java.util.List;
>-
> class A {
> 	void x() {
> 		long much = (long) (1 + 1) * Integer.MAX_VALUE;
>Index: resources/InlineTemp/canInline/A_test41_in.java
>===================================================================
>RCS file: resources/InlineTemp/canInline/A_test41_in.java
>diff -N resources/InlineTemp/canInline/A_test41_in.java
>--- /dev/null	1 Jan 1970 00:00:00 -0000
>+++ resources/InlineTemp/canInline/A_test41_in.java	1 Jan 1970 00:00:00 -0000
>@@ -0,0 +1,8 @@
>+package p;
>+
>+class A {
>+	void foo() {
>+		int a = 10 + 20;
>+		int b = 1 + a;
>+	}
>+}
>\ No newline at end of file
>Index: resources/InlineTemp/canInline/A_test41_out.java
>===================================================================
>RCS file: resources/InlineTemp/canInline/A_test41_out.java
>diff -N resources/InlineTemp/canInline/A_test41_out.java
>--- /dev/null	1 Jan 1970 00:00:00 -0000
>+++ resources/InlineTemp/canInline/A_test41_out.java	1 Jan 1970 00:00:00 -0000
>@@ -0,0 +1,7 @@
>+package p;
>+
>+class A {
>+	void foo() {
>+		int b = 1 + 10 + 20;
>+	}
>+}
>\ No newline at end of file
>Index: resources/InlineTemp/canInline/A_test42_in.java
>===================================================================
>RCS file: resources/InlineTemp/canInline/A_test42_in.java
>diff -N resources/InlineTemp/canInline/A_test42_in.java
>--- /dev/null	1 Jan 1970 00:00:00 -0000
>+++ resources/InlineTemp/canInline/A_test42_in.java	1 Jan 1970 00:00:00 -0000
>@@ -0,0 +1,8 @@
>+package p;
>+
>+class A {
>+	void foo() {
>+		int a = 10 + 20;
>+		int b = 1 - a;
>+	}
>+}
>\ No newline at end of file
>Index: resources/InlineTemp/canInline/A_test42_out.java
>===================================================================
>RCS file: resources/InlineTemp/canInline/A_test42_out.java
>diff -N resources/InlineTemp/canInline/A_test42_out.java
>--- /dev/null	1 Jan 1970 00:00:00 -0000
>+++ resources/InlineTemp/canInline/A_test42_out.java	1 Jan 1970 00:00:00 -0000
>@@ -0,0 +1,7 @@
>+package p;
>+
>+class A {
>+	void foo() {
>+		int b = 1 - (10 + 20);
>+	}
>+}
>\ No newline at end of file
>Index: resources/InlineTemp/canInline/A_test43_in.java
>===================================================================
>RCS file: resources/InlineTemp/canInline/A_test43_in.java
>diff -N resources/InlineTemp/canInline/A_test43_in.java
>--- /dev/null	1 Jan 1970 00:00:00 -0000
>+++ resources/InlineTemp/canInline/A_test43_in.java	1 Jan 1970 00:00:00 -0000
>@@ -0,0 +1,8 @@
>+package p;
>+
>+class A {
>+	void x() {
>+		long two = 2 * Integer.MAX_VALUE;
>+		long much = two * Integer.MAX_VALUE;
>+	}
>+}
>Index: resources/InlineTemp/canInline/A_test43_out.java
>===================================================================
>RCS file: resources/InlineTemp/canInline/A_test43_out.java
>diff -N resources/InlineTemp/canInline/A_test43_out.java
>--- /dev/null	1 Jan 1970 00:00:00 -0000
>+++ resources/InlineTemp/canInline/A_test43_out.java	1 Jan 1970 00:00:00 -0000
>@@ -0,0 +1,7 @@
>+package p;
>+
>+class A {
>+	void x() {
>+		long much = (long) (2 * Integer.MAX_VALUE) * Integer.MAX_VALUE;
>+	}
>+}
>Index: resources/IntroduceParameter/simple/Expression1.java
>===================================================================
>RCS file: resources/IntroduceParameter/simple/Expression1.java
>diff -N resources/IntroduceParameter/simple/Expression1.java
>--- /dev/null	1 Jan 1970 00:00:00 -0000
>+++ resources/IntroduceParameter/simple/Expression1.java	1 Jan 1970 00:00:00 -0000
>@@ -0,0 +1,12 @@
>+//selection: 7, 21, 7, 26
>+//name: i -> first
>+package simple;
>+
>+public class Expression1 {
>+	public void m() {
>+		int a= 1 - (2 + 3);
>+	}
>+	public void use() {
>+		m();
>+	}
>+}
>Index: resources/IntroduceParameter/simple/Expression2.java
>===================================================================
>RCS file: resources/IntroduceParameter/simple/Expression2.java
>diff -N resources/IntroduceParameter/simple/Expression2.java
>--- /dev/null	1 Jan 1970 00:00:00 -0000
>+++ resources/IntroduceParameter/simple/Expression2.java	1 Jan 1970 00:00:00 -0000
>@@ -0,0 +1,12 @@
>+//selection: 7, 20, 7, 27
>+//name: i -> first
>+package simple;
>+
>+public class Expression2 {
>+	public void m() {
>+		int a= 1 - (2 + 3);
>+	}
>+	public void use() {
>+		m();
>+	}
>+}
>Index: resources/IntroduceParameter/simple/Expression3.java
>===================================================================
>RCS file: resources/IntroduceParameter/simple/Expression3.java
>diff -N resources/IntroduceParameter/simple/Expression3.java
>--- /dev/null	1 Jan 1970 00:00:00 -0000
>+++ resources/IntroduceParameter/simple/Expression3.java	1 Jan 1970 00:00:00 -0000
>@@ -0,0 +1,12 @@
>+//selection: 7, 22, 7, 27
>+//name: i -> first
>+package simple;
>+
>+public class Expression3 {
>+	public void m() {
>+		int a= 1 - ((2 + 3));
>+	}
>+	public void use() {
>+		m();
>+	}
>+}
>Index: resources/IntroduceParameter/simple/Expression4.java
>===================================================================
>RCS file: resources/IntroduceParameter/simple/Expression4.java
>diff -N resources/IntroduceParameter/simple/Expression4.java
>--- /dev/null	1 Jan 1970 00:00:00 -0000
>+++ resources/IntroduceParameter/simple/Expression4.java	1 Jan 1970 00:00:00 -0000
>@@ -0,0 +1,12 @@
>+//selection: 7, 21, 7, 28
>+//name: i -> first
>+package simple;
>+
>+public class Expression4 {
>+	public void m() {
>+		int a= 1 - ((2 + 3));
>+	}
>+	public void use() {
>+		m();
>+	}
>+}
>Index: resources/IntroduceParameter/simple/Expression5.java
>===================================================================
>RCS file: resources/IntroduceParameter/simple/Expression5.java
>diff -N resources/IntroduceParameter/simple/Expression5.java
>--- /dev/null	1 Jan 1970 00:00:00 -0000
>+++ resources/IntroduceParameter/simple/Expression5.java	1 Jan 1970 00:00:00 -0000
>@@ -0,0 +1,12 @@
>+//selection: 7, 20, 7, 29
>+//name: i -> first
>+package simple;
>+
>+public class Expression5 {
>+	public void m() {
>+		int a= 1 - ((2 + 3));
>+	}
>+	public void use() {
>+		m();
>+	}
>+}
>Index: resources/IntroduceParameter/simple/out/Expression1.java
>===================================================================
>RCS file: resources/IntroduceParameter/simple/out/Expression1.java
>diff -N resources/IntroduceParameter/simple/out/Expression1.java
>--- /dev/null	1 Jan 1970 00:00:00 -0000
>+++ resources/IntroduceParameter/simple/out/Expression1.java	1 Jan 1970 00:00:00 -0000
>@@ -0,0 +1,12 @@
>+//selection: 7, 21, 7, 26
>+//name: i -> first
>+package simple.out;
>+
>+public class Expression1 {
>+	public void m(int first) {
>+		int a= 1 - first;
>+	}
>+	public void use() {
>+		m(2 + 3);
>+	}
>+}
>Index: resources/IntroduceParameter/simple/out/Expression2.java
>===================================================================
>RCS file: resources/IntroduceParameter/simple/out/Expression2.java
>diff -N resources/IntroduceParameter/simple/out/Expression2.java
>--- /dev/null	1 Jan 1970 00:00:00 -0000
>+++ resources/IntroduceParameter/simple/out/Expression2.java	1 Jan 1970 00:00:00 -0000
>@@ -0,0 +1,12 @@
>+//selection: 7, 20, 7, 27
>+//name: i -> first
>+package simple.out;
>+
>+public class Expression2 {
>+	public void m(int first) {
>+		int a= 1 - first;
>+	}
>+	public void use() {
>+		m(2 + 3);
>+	}
>+}
>Index: resources/IntroduceParameter/simple/out/Expression3.java
>===================================================================
>RCS file: resources/IntroduceParameter/simple/out/Expression3.java
>diff -N resources/IntroduceParameter/simple/out/Expression3.java
>--- /dev/null	1 Jan 1970 00:00:00 -0000
>+++ resources/IntroduceParameter/simple/out/Expression3.java	1 Jan 1970 00:00:00 -0000
>@@ -0,0 +1,12 @@
>+//selection: 7, 22, 7, 27
>+//name: i -> first
>+package simple.out;
>+
>+public class Expression3 {
>+	public void m(int first) {
>+		int a= 1 - (first);
>+	}
>+	public void use() {
>+		m(2 + 3);
>+	}
>+}
>Index: resources/IntroduceParameter/simple/out/Expression4.java
>===================================================================
>RCS file: resources/IntroduceParameter/simple/out/Expression4.java
>diff -N resources/IntroduceParameter/simple/out/Expression4.java
>--- /dev/null	1 Jan 1970 00:00:00 -0000
>+++ resources/IntroduceParameter/simple/out/Expression4.java	1 Jan 1970 00:00:00 -0000
>@@ -0,0 +1,12 @@
>+//selection: 7, 21, 7, 28
>+//name: i -> first
>+package simple.out;
>+
>+public class Expression4 {
>+	public void m(int first) {
>+		int a= 1 - first;
>+	}
>+	public void use() {
>+		m(2 + 3);
>+	}
>+}
>Index: resources/IntroduceParameter/simple/out/Expression5.java
>===================================================================
>RCS file: resources/IntroduceParameter/simple/out/Expression5.java
>diff -N resources/IntroduceParameter/simple/out/Expression5.java
>--- /dev/null	1 Jan 1970 00:00:00 -0000
>+++ resources/IntroduceParameter/simple/out/Expression5.java	1 Jan 1970 00:00:00 -0000
>@@ -0,0 +1,12 @@
>+//selection: 7, 20, 7, 29
>+//name: i -> first
>+package simple.out;
>+
>+public class Expression5 {
>+	public void m(int first) {
>+		int a= 1 - first;
>+	}
>+	public void use() {
>+		m((2 + 3));
>+	}
>+}
>Index: resources/SefWorkSpace/SefTests/object_in/TestCompoundWrite4.java
>===================================================================
>RCS file: resources/SefWorkSpace/SefTests/object_in/TestCompoundWrite4.java
>diff -N resources/SefWorkSpace/SefTests/object_in/TestCompoundWrite4.java
>--- /dev/null	1 Jan 1970 00:00:00 -0000
>+++ resources/SefWorkSpace/SefTests/object_in/TestCompoundWrite4.java	1 Jan 1970 00:00:00 -0000
>@@ -0,0 +1,9 @@
>+package object_in;
>+
>+public class TestCompoundWrite4 {
>+	int field;
>+	
>+	public void foo() {
>+		field+= 1 + 2;
>+	}
>+}
>Index: resources/SefWorkSpace/SefTests/object_out/TestCompoundWrite4.java
>===================================================================
>RCS file: resources/SefWorkSpace/SefTests/object_out/TestCompoundWrite4.java
>diff -N resources/SefWorkSpace/SefTests/object_out/TestCompoundWrite4.java
>--- /dev/null	1 Jan 1970 00:00:00 -0000
>+++ resources/SefWorkSpace/SefTests/object_out/TestCompoundWrite4.java	1 Jan 1970 00:00:00 -0000
>@@ -0,0 +1,17 @@
>+package object_out;
>+
>+public class TestCompoundWrite4 {
>+	private int field;
>+	
>+	public void foo() {
>+		setField(getField() + (1 + 2));
>+	}
>+
>+	int getField() {
>+		return field;
>+	}
>+
>+	void setField(int field) {
>+		this.field = field;
>+	}
>+}
>Index: test cases/org/eclipse/jdt/ui/tests/refactoring/ExtractConstantTests.java
>===================================================================
>RCS file: /cvsroot/eclipse/org.eclipse.jdt.ui.tests.refactoring/test cases/org/eclipse/jdt/ui/tests/refactoring/ExtractConstantTests.java,v
>retrieving revision 1.51
>diff -u -r1.51 ExtractConstantTests.java
>--- test cases/org/eclipse/jdt/ui/tests/refactoring/ExtractConstantTests.java	18 Jun 2010 17:55:11 -0000	1.51
>+++ test cases/org/eclipse/jdt/ui/tests/refactoring/ExtractConstantTests.java	21 Feb 2011 04:10:22 -0000
>@@ -1,5 +1,5 @@
> /*******************************************************************************
>- * Copyright (c) 2000, 2010 IBM Corporation and others.
>+ * Copyright (c) 2000, 2011 IBM Corporation and others.
> * All rights reserved. This program and the accompanying materials
> * are made available under the terms of the Eclipse Public License v1.0
> * which accompanies this distribution, and is available at
>@@ -325,6 +325,26 @@
> 		helper1(3, 19, 3, 24, true, false, "S_ALL", "ALL");
> 	}
> 
>+	public void test39() throws Exception { // bug 335173
>+		helper1(5, 21, 5, 26, false, false, "CONSTANT", "INT");
>+	}
>+
>+	public void test40() throws Exception { // bug 335173
>+		helper1(5, 20, 5, 27, false, false, "CONSTANT", "INT");
>+	}
>+
>+	public void test41() throws Exception { // bug 335173
>+		helper1(5, 22, 5, 27, false, false, "CONSTANT", "INT");
>+	}
>+
>+	public void test42() throws Exception { // bug 335173
>+		helper1(5, 21, 5, 28, false, false, "CONSTANT", "INT");
>+	}
>+
>+	public void test43() throws Exception { // bug 335173
>+		helper1(5, 20, 5, 29, false, false, "CONSTANT", "INT");
>+	}
>+
> 	public void testZeroLengthSelection0() throws Exception {
> 		helper1(5, 18, 5, 18, false, false, "CONSTANT", "_100");
> 	}
>Index: test cases/org/eclipse/jdt/ui/tests/refactoring/ExtractMethodTests.java
>===================================================================
>RCS file: /cvsroot/eclipse/org.eclipse.jdt.ui.tests.refactoring/test cases/org/eclipse/jdt/ui/tests/refactoring/ExtractMethodTests.java,v
>retrieving revision 1.97
>diff -u -r1.97 ExtractMethodTests.java
>--- test cases/org/eclipse/jdt/ui/tests/refactoring/ExtractMethodTests.java	9 Nov 2010 04:55:04 -0000	1.97
>+++ test cases/org/eclipse/jdt/ui/tests/refactoring/ExtractMethodTests.java	21 Feb 2011 04:10:22 -0000
>@@ -1,5 +1,5 @@
> /*******************************************************************************
>- * Copyright (c) 2000, 2010 IBM Corporation and others.
>+ * Copyright (c) 2000, 2011 IBM Corporation and others.
> * All rights reserved. This program and the accompanying materials
> * are made available under the terms of the Eclipse Public License v1.0
> * which accompanies this distribution, and is available at
>@@ -1445,6 +1445,26 @@
> 		expressionTest();
> 	}
> 
>+	public void test625() throws Exception {
>+		expressionTest();
>+	}
>+
>+	public void test626() throws Exception {
>+		expressionTest();
>+	}
>+
>+	public void test627() throws Exception {
>+		expressionTest();
>+	}
>+
>+	public void test628() throws Exception {
>+		expressionTest();
>+	}
>+
>+	public void test629() throws Exception {
>+		expressionTest();
>+	}
>+
> 	//---- Test nested methods and constructor
> 
> 	public void test650() throws Exception {
>Index: test cases/org/eclipse/jdt/ui/tests/refactoring/ExtractTempTests.java
>===================================================================
>RCS file: /cvsroot/eclipse/org.eclipse.jdt.ui.tests.refactoring/test cases/org/eclipse/jdt/ui/tests/refactoring/ExtractTempTests.java,v
>retrieving revision 1.109
>diff -u -r1.109 ExtractTempTests.java
>--- test cases/org/eclipse/jdt/ui/tests/refactoring/ExtractTempTests.java	20 Apr 2010 19:42:15 -0000	1.109
>+++ test cases/org/eclipse/jdt/ui/tests/refactoring/ExtractTempTests.java	21 Feb 2011 04:10:22 -0000
>@@ -1,5 +1,5 @@
> /*******************************************************************************
>- * Copyright (c) 2000, 2010 IBM Corporation and others.
>+ * Copyright (c) 2000, 2011 IBM Corporation and others.
> * All rights reserved. This program and the accompanying materials
> * are made available under the terms of the Eclipse Public License v1.0
> * which accompanies this distribution, and is available at
>@@ -656,6 +656,31 @@
> 		helper1(6, 17, 6, 24, true, false, "temp", "i");
> 	}
> 
>+	public void test105() throws Exception {
>+		//test for https://bugs.eclipse.org/bugs/show_bug.cgi?id=335173
>+		helper1(5, 21, 5, 26, true, false, "temp", "i");
>+	}
>+
>+	public void test106() throws Exception {
>+		//test for https://bugs.eclipse.org/bugs/show_bug.cgi?id=335173
>+		helper1(5, 20, 5, 27, true, false, "temp", "i");
>+	}
>+
>+	public void test107() throws Exception {
>+		//test for https://bugs.eclipse.org/bugs/show_bug.cgi?id=335173
>+		helper1(5, 22, 5, 27, true, false, "temp", "i");
>+	}
>+
>+	public void test108() throws Exception {
>+		//test for https://bugs.eclipse.org/bugs/show_bug.cgi?id=335173
>+		helper1(5, 21, 5, 28, true, false, "temp", "i");
>+	}
>+
>+	public void test109() throws Exception {
>+		//test for https://bugs.eclipse.org/bugs/show_bug.cgi?id=335173
>+		helper1(5, 20, 5, 29, true, false, "temp", "i");
>+	}
>+
> 	public void testZeroLengthSelection0() throws Exception {
> //		printTestDisabledMessage("test for bug 30146");
> 		helper1(4, 18, 4, 18, true, false, "temp", "j");
>Index: test cases/org/eclipse/jdt/ui/tests/refactoring/InlineConstantTests.java
>===================================================================
>RCS file: /cvsroot/eclipse/org.eclipse.jdt.ui.tests.refactoring/test cases/org/eclipse/jdt/ui/tests/refactoring/InlineConstantTests.java,v
>retrieving revision 1.36
>diff -u -r1.36 InlineConstantTests.java
>--- test cases/org/eclipse/jdt/ui/tests/refactoring/InlineConstantTests.java	19 Jan 2010 14:09:07 -0000	1.36
>+++ test cases/org/eclipse/jdt/ui/tests/refactoring/InlineConstantTests.java	21 Feb 2011 04:10:22 -0000
>@@ -1,5 +1,5 @@
> /*******************************************************************************
>- * Copyright (c) 2000, 2010 IBM Corporation and others.
>+ * Copyright (c) 2000, 2011 IBM Corporation and others.
> * All rights reserved. This program and the accompanying materials
> * are made available under the terms of the Eclipse Public License v1.0
> * which accompanies this distribution, and is available at
>@@ -300,6 +300,18 @@
> 		helper1("p.A", 4, 24, 4, 25, true, true);
> 	}
> 	
>+	public void test35() throws Exception { // test for https://bugs.eclipse.org/bugs/show_bug.cgi?id=335173
>+		helper1("p.A", 4, 22, 4, 23, true, true);
>+	}
>+
>+	public void test36() throws Exception { // test for https://bugs.eclipse.org/bugs/show_bug.cgi?id=335173
>+		helper1("p.A", 4, 22, 4, 23, true, true);
>+	}
>+
>+	public void test37() throws Exception { // test for https://bugs.eclipse.org/bugs/show_bug.cgi?id=335173
>+		helper1("p.A", 4, 23, 4, 28, true, true);
>+	}
>+
> 	// -- testing failing preconditions
> 
> 	public void testFail0() throws Exception {
>Index: test cases/org/eclipse/jdt/ui/tests/refactoring/InlineMethodTests.java
>===================================================================
>RCS file: /cvsroot/eclipse/org.eclipse.jdt.ui.tests.refactoring/test cases/org/eclipse/jdt/ui/tests/refactoring/InlineMethodTests.java,v
>retrieving revision 1.89
>diff -u -r1.89 InlineMethodTests.java
>--- test cases/org/eclipse/jdt/ui/tests/refactoring/InlineMethodTests.java	17 Nov 2010 05:26:26 -0000	1.89
>+++ test cases/org/eclipse/jdt/ui/tests/refactoring/InlineMethodTests.java	21 Feb 2011 04:10:22 -0000
>@@ -1,5 +1,5 @@
> /*******************************************************************************
>- * Copyright (c) 2000, 2010 IBM Corporation and others.
>+ * Copyright (c) 2000, 2011 IBM Corporation and others.
> * All rights reserved. This program and the accompanying materials
> * are made available under the terms of the Eclipse Public License v1.0
> * which accompanies this distribution, and is available at
>@@ -836,6 +836,14 @@
> 		performCastTest();
> 	}
> 
>+	public void testInfixExpression1() throws Exception {
>+		performCastTest();
>+	}
>+
>+	public void testInfixExpression2() throws Exception {
>+		performCastTest();
>+	}
>+
> 	/* *********************** Enum Tests ******************************* */
> 
> 	private void performEnumTest() throws Exception {
>@@ -948,6 +956,10 @@
> 		performOperatorTest();
> 	}
> 
>+	public void testPlusPlus_1() throws Exception {
>+		performOperatorTest();
>+	}
>+
> 	public void testPlusDiff() throws Exception {
> 		performOperatorTest();
> 	}
>@@ -956,6 +968,10 @@
> 		performOperatorTest();
> 	}
> 
>+	public void testDiffPlus() throws Exception {
>+		performOperatorTest();
>+	}
>+
> 	public void testTimesPlus() throws Exception {
> 		performOperatorTest();
> 	}
>Index: test cases/org/eclipse/jdt/ui/tests/refactoring/InlineTempTests.java
>===================================================================
>RCS file: /cvsroot/eclipse/org.eclipse.jdt.ui.tests.refactoring/test cases/org/eclipse/jdt/ui/tests/refactoring/InlineTempTests.java,v
>retrieving revision 1.60
>diff -u -r1.60 InlineTempTests.java
>--- test cases/org/eclipse/jdt/ui/tests/refactoring/InlineTempTests.java	19 Jan 2010 14:09:07 -0000	1.60
>+++ test cases/org/eclipse/jdt/ui/tests/refactoring/InlineTempTests.java	21 Feb 2011 04:10:22 -0000
>@@ -1,5 +1,5 @@
> /*******************************************************************************
>- * Copyright (c) 2000, 2010 IBM Corporation and others.
>+ * Copyright (c) 2000, 2011 IBM Corporation and others.
> * All rights reserved. This program and the accompanying materials
> * are made available under the terms of the Eclipse Public License v1.0
> * which accompanies this distribution, and is available at
>@@ -277,7 +277,7 @@
> 	
> 	public void test36() throws Exception{
> 		// parenthesize complex cast expression
>-		helper1(8, 21, 8, 24);
>+		helper1(6, 21, 6, 24);
> 	}
> 	
> 	public void test37() throws Exception{
>@@ -300,6 +300,21 @@
> 		helper1(5, 43, 5, 46);
> 	}
> 	
>+	public void test41() throws Exception {
>+		// https://bugs.eclipse.org/bugs/show_bug.cgi?id=335173
>+		helper1(5, 13, 5, 14);
>+	}
>+
>+	public void test42() throws Exception {
>+		// https://bugs.eclipse.org/bugs/show_bug.cgi?id=335173
>+		helper1(5, 13, 5, 14);
>+	}
>+
>+	public void test43() throws Exception {
>+		// parenthesize complex cast expression https://bugs.eclipse.org/bugs/show_bug.cgi?id=335173
>+		helper1(6, 21, 6, 24);
>+	}
>+
> 	//------
> 
> 	public void testFail0() throws Exception{
>Index: test cases/org/eclipse/jdt/ui/tests/refactoring/IntroduceParameterTests.java
>===================================================================
>RCS file: /cvsroot/eclipse/org.eclipse.jdt.ui.tests.refactoring/test cases/org/eclipse/jdt/ui/tests/refactoring/IntroduceParameterTests.java,v
>retrieving revision 1.20
>diff -u -r1.20 IntroduceParameterTests.java
>--- test cases/org/eclipse/jdt/ui/tests/refactoring/IntroduceParameterTests.java	31 Dec 2008 21:14:41 -0000	1.20
>+++ test cases/org/eclipse/jdt/ui/tests/refactoring/IntroduceParameterTests.java	21 Feb 2011 04:10:22 -0000
>@@ -1,5 +1,5 @@
> /*******************************************************************************
>- * Copyright (c) 2000, 2008 IBM Corporation and others.
>+ * Copyright (c) 2000, 2011 IBM Corporation and others.
> * All rights reserved. This program and the accompanying materials
> * are made available under the terms of the Eclipse Public License v1.0
> * which accompanies this distribution, and is available at
>@@ -153,6 +153,26 @@
> 		performOK();
> 	}
> 
>+	public void testSimple_Expression1() throws Exception {
>+		performOK();
>+	}
>+
>+	public void testSimple_Expression2() throws Exception {
>+		performOK();
>+	}
>+
>+	public void testSimple_Expression3() throws Exception {
>+		performOK();
>+	}
>+
>+	public void testSimple_Expression4() throws Exception {
>+		performOK();
>+	}
>+
>+	public void testSimple_Expression5() throws Exception {
>+		performOK();
>+	}
>+
> 	public void testSimple_NewInstance1() throws Exception {
> 		performOK();
> 	}
>Index: test cases/org/eclipse/jdt/ui/tests/refactoring/SefTests.java
>===================================================================
>RCS file: /cvsroot/eclipse/org.eclipse.jdt.ui.tests.refactoring/test cases/org/eclipse/jdt/ui/tests/refactoring/SefTests.java,v
>retrieving revision 1.37
>diff -u -r1.37 SefTests.java
>--- test cases/org/eclipse/jdt/ui/tests/refactoring/SefTests.java	12 Nov 2008 10:26:59 -0000	1.37
>+++ test cases/org/eclipse/jdt/ui/tests/refactoring/SefTests.java	21 Feb 2011 04:10:22 -0000
>@@ -1,5 +1,5 @@
> /*******************************************************************************
>- * Copyright (c) 2000, 2008 IBM Corporation and others.
>+ * Copyright (c) 2000, 2011 IBM Corporation and others.
> * All rights reserved. This program and the accompanying materials
> * are made available under the terms of the Eclipse Public License v1.0
> * which accompanies this distribution, and is available at
>@@ -238,6 +238,10 @@
> 		objectTest("field");
> 	}
> 
>+	public void testCompoundWrite4() throws Exception {
>+		objectTest("field");
>+	}
>+
> 	public void testFinalField() throws Exception {
> 		objectTest("field");
> 	}

Actions: View | Diff

Attachments on bug 335173: 187952 | 188224 | 188311 | 188316 | 188426 | 189008 | 189072 | 189172 | 189197 | 189391