[webkit-reviews] review denied: [Bug 5874] WebCore+SVG needs support for "evt" instead of event : [Attachment 5836] addressed Darin's concerns

bugzilla-request-daemon at opendarwin.org bugzilla-request-daemon at opendarwin.org
Sun Jan 22 10:46:21 PST 2006


Darin Adler <darin at apple.com> has denied Eric Seidel <macdome at opendarwin.org>'s
request for review:
Bug 5874: WebCore+SVG needs support for "evt" instead of event
http://bugzilla.opendarwin.org/show_bug.cgi?id=5874

Attachment 5836: addressed Darin's concerns
http://bugzilla.opendarwin.org/attachment.cgi?id=5836&action=edit

------- Additional Comments from Darin Adler <darin at apple.com>
I don't like the underscore-prefixed name for the local variable. We should
steer away from underscore prefixes in general since they are reserved for the
library and platform.

Patch does not include JSSVGLazyEventListener.h and .cpp so I couldn't review
them. Because of that, review-.



More information about the webkit-reviews mailing list