Bug 9057

Summary:

Add HasIndexGetter support to bindings autogeneration system

Product:

WebKit

Reporter:

Eric Seidel (no email) <eric>

Component:

WebCore Misc.

Assignee:

Eric Seidel (no email) <eric>

Status:

RESOLVED FIXED

Severity:

Normal

Priority:

Version:

420+

Hardware:

Mac

OS:

OS X 10.4

Attachments:

Description	Flags
First stab. This needs just a bit more testing, but posting for andersca's enjoyment.	none
Slightly better patch	andersca: review+
Even better patch	none
yet another patch for review	andersca: review+

Eric Seidel (no email)

Reported 2006-05-23 02:25:47 PDT

Add HasIndexGetter support to bindings autogeneration system One more autogen blocker out of the way...

Attachments
First stab. This needs just a bit more testing, but posting for andersca's enjoyment. (19.72 KB, patch) 2006-05-23 02:26 PDT, Eric Seidel (no email)	no flags	Details Formatted Diff Diff
Slightly better patch (20.31 KB, patch) 2006-05-23 02:32 PDT, Eric Seidel (no email)	andersca: review+	Details Formatted Diff Diff
Even better patch (58.77 KB, patch) 2006-05-24 17:22 PDT, Eric Seidel (no email)	no flags	Details Formatted Diff Diff
yet another patch for review (58.65 KB, patch) 2006-05-24 17:48 PDT, Eric Seidel (no email)	andersca: review+	Details Formatted Diff Diff
Show Obsolete (3) View All Add attachment proposed patch, testcase, etc.

Eric Seidel (no email)

Comment 1 2006-05-23 02:26:33 PDT

Created attachment 8476 [details] First stab. This needs just a bit more testing, but posting for andersca's enjoyment.

Eric Seidel (no email)

Comment 2 2006-05-23 02:32:27 PDT

Created attachment 8478 [details] Slightly better patch

Anders Carlsson

Comment 3 2006-05-23 09:25:21 PDT

Comment on attachment 8478 [details] Slightly better patch The patch looks good. I'd like to see some more testing of this mechanism with other classes, so we know that it's general enough (together with HasNameGetter)

Eric Seidel (no email)

Comment 4 2006-05-24 17:22:01 PDT

Created attachment 8528 [details] Even better patch

Eric Seidel (no email)

Comment 5 2006-05-24 17:48:09 PDT

Created attachment 8529 [details] yet another patch for review

Anders Carlsson

Comment 6 2006-05-25 14:29:36 PDT

Comment on attachment 8529 [details] yet another patch for review r=me

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