Summary: | calling delete on enum constant deletes entire Enum | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | [Eclipse Project] JDT | Reporter: | Vikas Trivedi <vtrivedi> | ||||||
Component: | Core | Assignee: | Olivier Thomann <Olivier_Thomann> | ||||||
Status: | VERIFIED FIXED | QA Contact: | |||||||
Severity: | blocker | ||||||||
Priority: | P3 | CC: | philippe_mulet | ||||||
Version: | 3.2 | ||||||||
Target Milestone: | 3.2.2 | ||||||||
Hardware: | PC | ||||||||
OS: | Windows XP | ||||||||
Whiteboard: | |||||||||
Attachments: |
|
Description
Vikas Trivedi
2006-09-14 11:29:14 EDT
I'll investigate. Created attachment 50217 [details]
Proposed fix
Created attachment 50218 [details]
Regression test
Philippe, This would be a good candidate for 3.2.2. Released for 3.3M2. Regression test added in org.eclipse.jdt.core.tests.model.DeleteTests#testDeleteField5. I'd like to request that this fix be pushed into 3.2.1. We have identified atleast 2 components (and possibly a couple more) that are impacted by this. And in general anyone attempting to get the ast node for the enum literal is going to be adversely affected. Without this fix, we have data loss (entire enum is deleted). Our final contribution to 3.2.1 has already been made. This defect was reported too late to be part of 3.2.1. Good candidate for 3.2.2. reopening to address in 3.2 maintenance stream (-> 3.2.2) Note that we may be able to post an early preview patch I noticed some other fixes are still going in ... (https://bugs.eclipse.org/bugs/show_bug.cgi?id=151743) so I'm wondering if we can squeeze this one in too? As I mentioned this is a bad break for us and without the fix in, we have to either pull out some functionality or switch to using jdt internal calls ... both highly undesirable options. The other bug you refer too was entered a long time ago, and planned carefully. We are very selective in issues addressed for a rollup, and near the end of the fixing window we have to refrain from late changes. This one is tempting, but just too late. As one said: "Better is the enemy of good" (Voltaire). Fair enough :). In order to get things working again at my end, I will be making calls to jdt internal code (JavaDeleteProcessor & Delete Wizard). Is there any risk of these changing or being removed any time soon? There is no risk for these to disappear within 3.2 maintenance stream. In 3.3 stream (and also in 3.2.2) we will address this very issue. Released for 3.2.2. Regression test added in org.eclipse.jdt.core.tests.model.DeleteTests#testDeleteField5 Verified for 3.3 M2 using build I200609220010. Verified for 3.2.2 using build M20070112-1200. (Running the regression test upon v_684_R32x indeed.) |