[webkit-reviews] review denied: [Bug 12878] Support for each (var in collection) syntax : [Attachment 15782] Updated Patch

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Wed Aug 1 10:25:05 PDT 2007


Mark Rowe (bdash) <mrowe at apple.com> has denied Justin Haygood
<jhaygood at spsu.edu>'s request for review:
Bug 12878: Support for each (var in collection) syntax
http://bugs.webkit.org/show_bug.cgi?id=12878

Attachment 15782: Updated Patch
http://bugs.webkit.org/attachment.cgi?id=15782&action=edit

------- Additional Comments from Mark Rowe (bdash) <mrowe at apple.com>
I pointed out a few issues in the patch to Justin via IRC.  There are numerous
coding style inconsistencies, and the redeclaration of ForInNode's instance
variables in the subclass is wrong.  I also suggested that the implementation
of streamTo could easily be shared between the two classes which should allow
ForInNode's members to remain private.



More information about the webkit-reviews mailing list