Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 354445 - Organize imports creates imports for eglx.lang... files
Summary: Organize imports creates imports for eglx.lang... files
Status: CLOSED FIXED
Alias: None
Product: z_Archived
Classification: Eclipse Foundation
Component: EDT (show other bugs)
Version: unspecified   Edit
Hardware: PC Windows XP
: P3 normal (vote)
Target Milestone: ---   Edit
Assignee: Tony Chen CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-08-10 18:03 EDT by broy2 CLA
Modified: 2017-02-23 14:14 EST (History)
4 users (show)

See Also:

Attachments
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 broy2 CLA 2011-08-10 18:03:36 EDT 
Build Identifier: 20110808 with IDE projects from IBM CVS on 0810

Organize imports creates imports for:
  import eglx.lang.basicprogram;
  import eglx.lang.syslib;

package pkg;

import eglx.lang.basicprogram;
import eglx.lang.syslib;

program pgm type BasicProgram {}
	const constantName string = "literalString";
	variableName string;
	
	function main()
		SysLib.writestdout(constantName);
	end	
end

Reproducible: Always

Steps to Reproduce:
1. Create a new program named pgm.
2. Add statement to main function:  SysLib.writestdout(constantName);
3. Organize imports.
4. 2 imports are added for eglx.. files.
Comment 1 Zhi Zhu CLA 2011-08-18 22:07:38 EDT 
Matt:
   This behavior occurs because EDT system now does not regard Syslib,BasicProgram as SysAnnotation.

   You can check from OrganizeImportsVisitor#addToResolvedTypes(IBinding binding) methods, now IAnnotationTypeBinding for Syslib,BasicProgram are not regarded as SysAnnotation.
Comment 2 Matt Heitz CLA 2011-08-19 09:43:58 EDT 
Zhi, the idea of "system parts" is probably going to be different in EDT than in RBD.  Paul has two enhancements (bug 354136 and bug 354137) which need to be completed before this problem can be fixed.  Those enhancements are targeted for iteration 13, so I changed the target of this bug for iteration 13.
Comment 3 broy2 CLA 2011-09-15 13:52:33 EDT 
Still a problem in 20110915. 
Waiting on fixes from Paul.  See comment.
Comment 4 Tony Chen CLA 2011-10-08 02:12:05 EDT 
The two enhancements (bug 354136 and bug 354137) are deferred, so this bug will have to be deferred as well. 

Moving it out of 0.7.
Comment 5 Tony Chen CLA 2012-04-05 03:59:01 EDT 
defer this since the dependents are deferred
Comment 6 Matt Heitz CLA 2012-11-12 14:18:32 EST 
I'm not sure how or when this was fixed, but it doesn't happen anymore.
Comment 7 Matt Heitz CLA 2013-01-03 10:43:39 EST 
Moving old fixed bugs from the RESOLVED state to CLOSED.