Summary: | [compiler] Improving debug strings for Break statement, IntLiteral and CaseStatement | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | [Eclipse Project] JDT | Reporter: | Elmar Chen <chxiao> | ||||||
Component: | Core | Assignee: | Olivier Thomann <Olivier_Thomann> | ||||||
Status: | VERIFIED FIXED | QA Contact: | |||||||
Severity: | trivial | ||||||||
Priority: | P3 | CC: | amj87.iitr, Olivier_Thomann | ||||||
Version: | 3.7 | ||||||||
Target Milestone: | 3.7 M2 | ||||||||
Hardware: | All | ||||||||
OS: | All | ||||||||
Whiteboard: | |||||||||
Attachments: |
|
Description
Elmar Chen
2010-09-09 08:43:38 EDT
Created attachment 178648 [details]
Proposed fix
Need to check that no test is impacted and also modified the break statement to display better.
Created attachment 178670 [details]
Proposed fix + regression tests
Updated all regression tests.
Released for 3.7M2. Updated existing regression tests. Olivier, its not clear to me why the constructor for IntLiteral and its printExpression were removed here. I dont see any clear correlation to this bug as such. Were they removed because they are not used anywhere? (In reply to comment #4) > Olivier, its not clear to me why the constructor for IntLiteral and its > printExpression were removed here. I dont see any clear correlation to this bug > as such. Were they removed because they are not used anywhere? Yes, we used to use this old constructor, but we don't anymore. So for clean up this I removed them. Verified for 3.7M2 using code inspection. |