Summary: | StackOverflowError in BinaryExpression | ||
---|---|---|---|
Product: | [Eclipse Project] JDT | Reporter: | David Audel <david_audel> |
Component: | Core | Assignee: | Olivier Thomann <Olivier_Thomann> |
Status: | VERIFIED FIXED | QA Contact: | |
Severity: | normal | ||
Priority: | P3 | CC: | jeem |
Version: | 3.0 | Keywords: | performance |
Target Milestone: | 3.0 M9 | ||
Hardware: | PC | ||
OS: | Windows 2000 | ||
Whiteboard: |
Description
David Audel
2002-06-11 09:58:27 EDT
Defer... DOM APIs should be protected. There is some potential for this to occur in outliner. The DOM AST isn't able to treat multi string concatenations in one node. e.g. String s = "aaa" + "aab" + "aac" + .... + "zzz"; It should perform similar optimization as compiler does. Actually, previous comment should have said that it has potential to occur in AST created during reconcile operation. InfixExpression.extendedOperands allows multi-string concatenations etc. to be represented as broad, flat ASTs. Olivier - pls verify that DOM AST had now support for handling these n-ary operations (should be InfixExpression.extendedOperands). Also check that DOM AST constructed during #reconcile operation is indeed building such a flavor of an AST. Suspecting this to cause the fact we cannot open the ASTView on OldSearchTests. Ask Frederic Fusier for exact steps to reproduce. You can get it also with ConformTest. To reproduce it: 1) open the ASTView (Window->Show View->Other...->Java->ASTView), 2) open the file (ConformTest.java or OldSearchTests.java), 3) click on ASTView bar button 'Show AST of active editor' => you get the StackOverflow exception... This is fixed and released in HEAD. AST view is still failing for OldSearchTests, because of the recursion done in the AST visitor, but it is fixed for ConformTests. Verified for 3.0M9 |