[webkit-reviews] review granted: [Bug 8813] Optimizing +[DOMNode
_nodeWith:] : [Attachment 8191] Optimization
bugzilla-request-daemon at opendarwin.org
bugzilla-request-daemon at opendarwin.org
Tue May 9 17:42:30 PDT 2006
Darin Adler <darin at apple.com> has granted Darin Adler <darin at apple.com>'s
request for review:
Bug 8813: Optimizing +[DOMNode _nodeWith:]
http://bugzilla.opendarwin.org/show_bug.cgi?id=8813
Attachment 8191: Optimization
http://bugzilla.opendarwin.org/attachment.cgi?id=8191&action=edit
------- Additional Comments from Darin Adler <darin at apple.com>
Looks great, r=me.
Some minor comments that should not prvent us landing this:
+static ObjcClassMap *objcClassMap;
The * should be next to the type rather than the variable.
I think ObjC is clearer than Objc.
+ // Populate it with DOM Element classes.
I would say "element" not "Element".
+ addObjcClassForTag(htmlTag, [DOMHTMLHtmlElement class]);
Sorting these alphabetically would make it easier to see if something is in the
list or not.
More information about the webkit-reviews
mailing list