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

Bug 357936

Summary: JS ctors: Dead code removal
Product: z_Archived Reporter: Scott Greer <greer>
Component: EDTAssignee: Project Inbox <edt.javascriptgen-inbox>
Status: CLOSED FIXED QA Contact:
Severity: minor    
Priority: P3 CC: jvincens, mheitz
Version: unspecified   
Target Milestone: ---   
Hardware: PC   
OS: Windows XP   
Whiteboard:

Description Scott Greer CLA 2011-09-16 08:49:31 EDT
Scan both the RT and the JS generator for the following and remove them:

1)  Occurrences of "new egl.eglx.lang.StrLib" or "new egl.eglx.lang.StringLib" in constructors

2)  this.jsrt$SysVar= new egl.egl.core.SysVar();   seems to serve no purpose and should be removed after confirming with Matt and Joe.
Comment 1 Scott Greer CLA 2011-09-22 15:55:04 EDT
Matt, Joe,

Either of you have any reasons not to proceed with these two changes?
Comment 2 Joseph Vincens CLA 2011-09-22 15:57:26 EDT
please proceed
Comment 3 Matt Heitz CLA 2011-09-22 16:24:03 EDT
Looks OK to me.
Comment 4 Scott Greer CLA 2011-09-26 17:59:04 EDT
Fixed.
Comment 5 Lisa Lasher CLA 2011-11-02 10:26:06 EDT
closing this defect for Scott