groovy

Syntax error thrown, but should be semantic error

Details

  • Type: Improvement Improvement
  • Status: Open Open
  • Priority: Minor Minor
  • Resolution: Unresolved
  • Affects Version/s: 1.6.3
  • Fix Version/s: 1.8.x, 2.x
  • Component/s: None
  • Labels:
  • Number of attachments :
    0

Description

This program causes the compiler to throw a syntax error. However it should be a semantic error.

public class B {
   public void Test() {
      new NotExist()   
   }
}

Activity

Ascending order - Click to sort in descending order
Hide
Permalink
Paul King added a comment -

The current error message is:

1 compilation error:

unable to resolve class NotExist 
 at line: 2, column: 22

what are you expecting?

Show
Paul King added a comment - The current error message is: 1 compilation error: unable to resolve class NotExist at line: 2, column: 22 what are you expecting?
Hide
Permalink
blackdrag blackdrag added a comment -

It is more an improvement instead of a bug. The reporting is fine for the command line compiler and for simple reporting to the console. But for IDEs we could improve the error reporting by for example letting the exception have a pointer to the ClassExpression in this case here. Also the message class that is used, is SyntaxErrorMessage, which is simply the wrong name for an error, that is no syntax error.

Show
blackdrag blackdrag added a comment - It is more an improvement instead of a bug. The reporting is fine for the command line compiler and for simple reporting to the console. But for IDEs we could improve the error reporting by for example letting the exception have a pointer to the ClassExpression in this case here. Also the message class that is used, is SyntaxErrorMessage, which is simply the wrong name for an error, that is no syntax error.
Hide
Permalink
Andrew Eisenberg added a comment - - edited

Moving this down to "minor" because this issue is really only bothersome, but doesn't prevent me from getting my work done.

And, as Jochen mentions, this is mainly to help with IDE support.

Show
Andrew Eisenberg added a comment - - edited Moving this down to "minor" because this issue is really only bothersome, but doesn't prevent me from getting my work done. And, as Jochen mentions, this is mainly to help with IDE support.

People

  • Assignee:
    Unassigned
    Reporter:
    Andrew Eisenberg
Vote (0)
Watch (1)

Dates

  • Created:
    Updated: