Summary: | [DOM] code that would definitely cause NPE if executed | ||||||
---|---|---|---|---|---|---|---|
Product: | [Eclipse Project] JDT | Reporter: | Stephan Herrmann <stephan.herrmann> | ||||
Component: | Core | Assignee: | Olivier Thomann <Olivier_Thomann> | ||||
Status: | VERIFIED FIXED | QA Contact: | |||||
Severity: | normal | ||||||
Priority: | P3 | CC: | Olivier_Thomann | ||||
Version: | 3.7 | ||||||
Target Milestone: | 3.7 M6 | ||||||
Hardware: | All | ||||||
OS: | All | ||||||
Whiteboard: | |||||||
Attachments: |
|
Description
Stephan Herrmann
2011-02-23 07:15:04 EST
In fact that code can never be run. I'll clean it up. Created attachment 189706 [details]
Proposed fix
(In reply to comment #1) > In fact that code can never be run. I had hoped so :) So the code was trying to protect against inconsistencies between the dom Type and the compiler TypeBinding (one being an array type while the other's not)? And by now we're confident that such inconsistency will never occur? Sounds good to me. (In reply to comment #3) > So the code was trying to protect against inconsistencies between the dom Type > and the compiler TypeBinding (one being an array type while the other's not)? > > And by now we're confident that such inconsistency will never occur? > Sounds good to me. I believe the code was over paranoiac. At that point, it can only be an array type with an array type binding. So I'll run all tests for safety and I'll release afterwards. Released for 3.7M6. (In reply to comment #4) > I believe the code was over paranoiac. At that point, it can only be an array > type with an array type binding. I filed bug 339226 for recovering the underlying tacit knowledge. Verified for 3.7M6 via code inspection. |