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

Bug 343908

Summary: Alias annotation not working
Product: z_Archived Reporter: Justin Spadea <jspadea>
Component: EDTAssignee: Project Inbox <edt.javagen-inbox>
Status: CLOSED FIXED QA Contact:
Severity: normal    
Priority: P3 CC: jeffdouglas, mheitz
Version: unspecified   
Target Milestone: ---   
Hardware: PC   
OS: Linux   
Whiteboard:

Description Justin Spadea CLA 2011-04-26 17:09:18 EDT
Build Identifier: 

The alias annotation isn't working for logic parts.

program myprog{alias="aliasTest"}
    function main()
    end
end

The code is using IEGLConstants.PROPERTY_ALIAS in several places (including JS generator), which has a value of "alias". However the annotation type name on the IR is "egl.core.Alias" so no result is found and the part's ID is used instead. A new constant should be defined somewhere. You can't just update the current constant because it's used by the bindings in core, which need it to remain unqualified.

Reproducible: Always
Comment 1 Matt Heitz CLA 2011-07-12 14:26:37 EDT
Is there a compelling reason to support the alias annotation in EDT?  I'd rather remove the references to IEGLConstants.PROPERTY_ALIAS from the generator.
Comment 2 Justin Spadea CLA 2011-07-12 14:31:25 EDT
I'm okay with not supporting it. If we ever do support this though, debug will need to be updated to get the AST for the file so that it can retrieve this value when setting breakpoints (it needs to tell JDT the generated class name).
Comment 3 Jeff Douglas CLA 2011-07-14 09:13:39 EDT
The probably reason that we have aliasing in the first place would have come from CSP/VAG because of COBOL and the mainframe's limitation to 8 characters. As we are not providing COBOL generation support (at this time, or possibly ever) then I'm okay with removing it.
I'll make the changes now.
Comment 4 Jeff Douglas CLA 2011-07-14 09:31:21 EDT
Changes now made to no longer use the annotation for alias
Comment 5 Justin Spadea CLA 2011-09-13 15:34:11 EDT
Closing