Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 343908 - Alias annotation not working
Summary: Alias annotation not working
Status: CLOSED FIXED
Alias: None
Product: z_Archived
Classification: Eclipse Foundation
Component: EDT (show other bugs)
Version: unspecified   Edit
Hardware: PC Linux
: P3 normal (vote)
Target Milestone: ---   Edit
Assignee: Project Inbox CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-04-26 17:09 EDT by Justin Spadea CLA
Modified: 2017-02-23 14:17 EST (History)
2 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
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