Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 354750 - Implement arrays as java.util.List objects, not our EglLists
Summary: Implement arrays as java.util.List objects, not our EglLists
Status: CLOSED FIXED
Alias: None
Product: z_Archived
Classification: Eclipse Foundation
Component: EDT (show other bugs)
Version: unspecified   Edit
Hardware: PC Windows XP
: P3 enhancement (vote)
Target Milestone: ---   Edit
Assignee: Matt Heitz CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-08-15 14:33 EDT by Matt Heitz CLA
Modified: 2017-02-23 14:20 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 Matt Heitz CLA 2011-08-15 14:33:09 EDT 
This will allow for much better interoperability between generated and handwritten code.

The interface for an array should be java.util.List.  The implementation class should be java.util.ArrayList.

Delete the runtime's egl.lang.EglList interface.  Keep org.eclipse.edt.runtime.java.egl.lang.EglList, but it should only contain static methods which implement the functions of EglList.

The first argument to those static methods will be the array.  They should throw a NullValueException if the array is null.
Comment 1 Jeff Douglas CLA 2011-08-16 08:43:19 EDT 
can't implement it this way because the list needs to carry around a maxsize value, which would be lost
Comment 2 Lisa Lasher CLA 2011-08-26 17:53:16 EDT 
changing to Enhancement, as this isn't really a defect.
Comment 3 Matt Heitz CLA 2011-09-07 17:38:39 EDT 
Since maxSize has bitten the dust, the enhancement is now possible.
Comment 4 Matt Heitz CLA 2011-09-08 13:34:02 EDT 
We need to let Justin know when we make this change, so he can update the debugger.
Comment 5 Matt Heitz CLA 2011-09-09 12:43:15 EDT 
Adding Justin to the CC list so he'll know when we make this change.
Comment 6 Matt Heitz CLA 2011-10-04 17:05:28 EDT 
The work is complete.
Comment 7 Matt Heitz CLA 2011-11-01 10:41:51 EDT 
Verified.