[webkit-reviews] review granted: [Bug 10791] Even More Objective-C DOM auto-generation cleanup : [Attachment 10479] patch

bugzilla-request-daemon at opendarwin.org bugzilla-request-daemon at opendarwin.org
Sat Sep 9 14:16:04 PDT 2006


Eric Seidel <macdome at opendarwin.org> has granted Eric Seidel
<macdome at opendarwin.org>'s request for review:
Bug 10791: Even More Objective-C DOM auto-generation cleanup
http://bugzilla.opendarwin.org/show_bug.cgi?id=10791

Attachment 10479: patch
http://bugzilla.opendarwin.org/attachment.cgi?id=10479&action=edit

------- Additional Comments from Eric Seidel <macdome at opendarwin.org>
My thoughts

1.  not a fan of exclude=JS, it seems to be used for Obj-C only bindings.  This
only hinders efforts to make other language bindings (like ruby, python, perl,
etc.)

2.  I don't think the xpath stuff should be included in this patch.

3.  xpath stuff shouldn't need OldStyleObjC, since it's never shipped.

Other than that the patch looks fine.  r=me  (assuming you remove the xpath
stuff)



More information about the webkit-reviews mailing list