Bug 15735

Summary:

remove GroupNode to simplify AST and speed up a bit

Product:

WebKit

Reporter:

Darin Adler <darin>

Component:

JavaScriptCore

Assignee:

Darin Adler <darin>

Status:

RESOLVED FIXED

Severity:

Enhancement

Priority:

Version:

523.x (Safari 3)

Hardware:

Mac

OS:

OS X 10.4

Attachments:

Description	Flags
patch, change log, new tests (passes all existing tests)	mjs: review+

Darin Adler

Reported 2007-10-28 08:06:46 PDT

Maciej suggested that I get rid of GroupNode. The hard part is keeping serialization working.

Attachments
patch, change log, new tests (passes all existing tests) (129.14 KB, patch) 2007-10-28 14:17 PDT, Darin Adler	mjs: review+	Details Formatted Diff Diff
View All Add attachment proposed patch, testcase, etc.

Darin Adler

Comment 1 2007-10-28 14:17:45 PDT

Created attachment 16919 [details] patch, change log, new tests (passes all existing tests)

Maciej Stachowiak

Comment 2 2007-10-28 15:59:50 PDT

Comment on attachment 16919 [details] patch, change log, new tests (passes all existing tests) r=me

Darin Adler

Comment 3 2007-10-28 16:09:13 PDT

Committed revision 27191.

Note You need to log in before you can comment on or make changes to this bug.