[GROOVY-6608] GroovyLexer#require(...) method execution may have a NULL parser

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 2.2.0
• Fix Version/s: 2.2.3
• Component/s: ast builder
• Assignee: Jochen Theodorou
• Reporter: Alex Boyko
• Created: 05/Mar/14 21:58
• Updated: 15/Oct/15 18:19
• Resolved: 11/Mar/14 09:54

Description

The issue if found in Groovy Eclipse.

Test Scenario:
def scanFolders(){
doThis(
~"(?i)^sometext$", // SMART TAB FAILS AFTER HERE.
// ESCAPE THE DOLLAR SIGN AND THE WORLD SPINS AGAIN
~"(?i)^vti", // Sub-folder exclusion Filter
null,
null
);
};

Press enter at the end of the line and observe NPE in the log. NPE occurs in the method due to the null parser field.

The body of GroovyLexer#require(...) has a reference to parser that may not be set all the time. Furthermore the TODO comment in the method body states that error reporting should be done without parser usage: "Direct to a common error handler, rather than through the parser".