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:	EDT	Assignee:	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

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

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

2011-09-22 15:57:26 EDT

please proceed

Comment 3 Matt Heitz

2011-09-22 16:24:03 EDT

Looks OK to me.

Comment 4 Scott Greer

2011-09-26 17:59:04 EDT

Fixed.

Comment 5 Lisa Lasher

2011-11-02 10:26:06 EDT

closing this defect for Scott