[#GROOVY-1643] Please use parser.getASTFactory().setASTNodeClass(GroovySourceAST.class) - jira.codehaus.org

Details

Type: Bug
Status: Closed
Priority: Minor
Resolution: Fixed
Affects Version/s: None
Fix Version/s: 1.6-rc-1, 1.5.8, 1.7-beta-1
Component/s: None
Labels:
None

Number of attachments :
0

Description

Please use
parser.getASTFactory().setASTNodeClass(GroovySourceAST.class);
instead of
parser.setASTNodeClass("org.codehaus.groovy.antlr.GroovySourceAST");
in the JavaRecognizer and GroovyRecognizer because else it breakes the classloader in some cases.

Thanks
Richard Musiol

PS: The help icon right to this description box in "Create Issue" doesn't work.

Activity

Ascending order - Click to sort in descending order

Hide

Permalink

Andres Almiray added a comment - 03/Dec/07 9:19 PM

Could you provide a testcase showing why it won't work? thanks.

Show

Andres Almiray added a comment - 03/Dec/07 9:19 PM Could you provide a testcase showing why it won't work? thanks.

Hide

Permalink

Richard Musiol added a comment - 04/Dec/07 9:33 AM

It is quite a long time ago that I posted this bug, but as far as I remember the problem occurred because I used GROOVY inside an Eclipse plugin. The heavily manipulated classloader of Eclipse's plugin system could not find the class in some cases when the name was provided as a String. (Nevertheless it is also bad coding style to specify a class this way.)

Show

Richard Musiol added a comment - 04/Dec/07 9:33 AM It is quite a long time ago that I posted this bug, but as far as I remember the problem occurred because I used GROOVY inside an Eclipse plugin. The heavily manipulated classloader of Eclipse's plugin system could not find the class in some cases when the name was provided as a String. (Nevertheless it is also bad coding style to specify a class this way.)

People

Assignee:

blackdrag blackdrag

Reporter:

Richard Musiol

Vote (0)

Watch (0)

Dates

Created:

10/Jan/07 8:46 AM

Updated:

15/Oct/08 8:54 AM

Resolved:

15/Oct/08 8:54 AM