Bug 148575 - "Organize Imports" leaves warning markers untouched
Summary: "Organize Imports" leaves warning markers untouched
Status: RESOLVED INVALID
Alias: None
Product: JDT
Classification: Eclipse Project
Component: Text (show other bugs)
Version: 3.2   Edit
Hardware: Macintosh Mac OS X - Carbon (unsup.)
: P3 major (vote)
Target Milestone: ---   Edit
Assignee: JDT-Text-Inbox CLA
QA Contact:
URL:
Whiteboard:
Keywords: needinfo
Depends on:
Blocks:
 
Reported: 2006-06-25 15:54 EDT by Gerd Castan CLA
Modified: 2007-06-22 10:04 EDT (History)
1 user (show)

See Also:

Attachments
Before Organize Imports (173.42 KB, image/tiff)
2006-06-25 15:55 EDT, Gerd Castan CLA 		no flags Details
After Organize Imports (135.33 KB, image/tiff)
2006-06-25 15:56 EDT, Gerd Castan CLA 		no flags Details
After second Organize Imports (110.15 KB, image/tiff)
2006-06-25 15:56 EDT, Gerd Castan CLA 		no flags Details
After third Organize Imports (73.28 KB, image/tiff)
2006-06-25 15:57 EDT, Gerd Castan CLA 		no flags Details
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Gerd Castan CLA 2006-06-25 15:54:42 EDT 
This ist Callisto RC4 on Mac OS X

I have a class file with several unused imports.

"Organize Imports" removes the unused Imports but leaves the warning markers of the unused imports where they were. 

A second "Organize Imports" creates a complete mess. More "Organize Imports" are even more creative.

See Attachments.
Comment 1 Gerd Castan CLA 2006-06-25 15:55:33 EDT 
Created attachment 45265 [details]
Before Organize Imports
Comment 2 Gerd Castan CLA 2006-06-25 15:56:14 EDT 
Created attachment 45266 [details]
After Organize Imports
Comment 3 Gerd Castan CLA 2006-06-25 15:56:58 EDT 
Created attachment 45267 [details]
After second Organize Imports
Comment 4 Gerd Castan CLA 2006-06-25 15:57:35 EDT 
Created attachment 45268 [details]
After third Organize Imports
Comment 5 Olivier Thomann CLA 2006-06-25 22:45:44 EDT 
You need to save and rebuild your class to see the markers removed.
Moving to JDT/UI
Comment 6 Gerd Castan CLA 2006-06-26 01:31:34 EDT 
I tested again and this time,the markers are removed (without saving).

The the things I did between the bug report and this second test is sleeping and rebooting the computer, no development.
Comment 7 Martin Aeschlimann CLA 2006-06-26 06:10:16 EDT 
Look like the editor didn't call reconcile after the change. It also kept the old AST, resulting in the messed up code.
Comment 8 Dani Megert CLA 2006-06-26 06:13:45 EDT 
> It also kept the old AST, resulting in the messed up code.
Did you reproduce this?
Comment 9 Martin Aeschlimann CLA 2006-06-26 06:28:20 EDT 
I just made a guess. Didn't reproduce.
Comment 10 Dani Megert CLA 2006-06-26 06:37:34 EDT 
Works for me using 3.2 RC7 and I20060620-1010. Please see also:
http://www.eclipse.org/eclipse/platform-text/development/bug-incomplete.htm
Comment 11 Dani Megert CLA 2007-06-22 09:58:58 EDT 
Get rid of deprecated state.
Comment 12 Dani Megert CLA 2007-06-22 10:04:19 EDT 
.