Bug 16860

Summary:

a bit of cleanup after the Activation optimization

Product:

WebKit

Reporter:

Darin Adler <darin>

Component:

JavaScriptCore

Assignee:

Darin Adler <darin>

Status:

RESOLVED FIXED

Severity:

Normal

Priority:

Version:

528+ (Nightly build)

Hardware:

Mac

OS:

OS X 10.4

Attachments:

Description	Flags
patch	none
patch (revised with a tiny change to pass all tests)	eric: review+

Description Darin Adler 2008-01-13 09:17:40 PST

Here's a bit of clean-up fro the just-landed Activation optimization.

Comment 1 Darin Adler 2008-01-13 09:34:01 PST

Created attachment 18422 [details]
patch

Comment 2 Darin Adler 2008-01-13 11:18:41 PST

Created attachment 18424 [details]
patch (revised with a tiny change to pass all tests)

Comment 3 Eric Seidel (no email) 2008-01-25 13:09:24 PST

Comment on attachment 18424 [details]
patch (revised with a tiny change to pass all tests)

LGTM

Comment 4 Darin Adler 2008-01-25 13:10:35 PST

Comment on attachment 18424 [details]
patch (revised with a tiny change to pass all tests)

I think this patch no longer cleanly applies because of the new activation stack. I'll find out when I get some time.

Comment 5 Darin Adler 2008-01-26 10:58:31 PST

Committed revision 29810.