[webkit-reviews] review denied: [Bug 12878] Support for each ( var in collection ) syntax : [Attachment 13364] Adds support for for..each..in

[bugzilla-request-daemon at macosforge.org]

Alexey Proskuryakov <ap at webkit.org> has denied Alexey Proskuryakov
<ap at webkit.org>'s request for review:
Bug 12878: Support for each ( var in collection ) syntax
http://bugs.webkit.org/show_bug.cgi?id=12878

Attachment 13364: Adds support for for..each..in
http://bugs.webkit.org/attachment.cgi?id=13364&action=edit

------- Additional Comments from Alexey Proskuryakov <ap at webkit.org>
r- for purely stylistic issues discussed via IRC (star and comma positioning,
assert vs. ASSERT). And clearly, this cannot be landed to trunk before the
stabilization period ends.

A couple additional stylistic notes:

+	 if (varDecl)
+    s << "var " << varDecl;

  Indentation looks broken here.

+	     if ((c.complType() == Break) && ls.contains(c.target()))
+	     break;

  And here.

+    JSValue* e;
...
+    e = expr->evaluate(exec);

  There is no need to declare local variables at the beginning of the function
in C++. Also, this and other variables may benefit from a more descriptive
name.