Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

Bug 354445

Summary: Organize imports creates imports for eglx.lang... files
Product: z_Archived Reporter: broy2
Component: EDTAssignee: Tony Chen <chenzhh>
Status: CLOSED FIXED QA Contact:
Severity: normal    
Priority: P3 CC: chenzhh, mheitz, pharmon, svihovec
Version: unspecified   
Target Milestone: ---   
Hardware: PC   
OS: Windows XP   
Whiteboard:

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.