Description Chris West (Faux) 2010-05-20 18:47:08 EDT

Build Identifier: M20100211-1343

If you have a parenthesized expression (ParenthesizedExpression) containing just a variable/field reference (SimpleName), and that local variable is in error (undeclared, possibly uninitialised, etc.), the problem is reported for the entire expression, instead of the expected name.

From a UI perspective, this shows as, in:

class Scratch { void foo(int q) {
 foo(
  (
//^ here
   a
  )
//^ to here
 );
}}

... the entire section between the indicated brackets being underlined in red, including the comment.


From an API consumer point of view, the IProblem#getStartPosition() does not refer to the same point as the SimpleName#getStartPosition() for that variable error.

Workaround for consumers:
private int getRealErrorReportPosition(final SimpleName sn) {
 ASTNode test = sn;
 while (test.getParent() instanceof ParenthesizedExpression)
  test = test.getParent();
 return test.getStartPosition();
}


Reproducible: Always

Steps to Reproduce:
1. Paste the above class into a Java editor.
2. Note that "a" is in error, but the red-underlining for the error situation extends from the (unnecessary) open bracket to the close bracket.