<br><div class="gmail_quote">On Wed, Jan 23, 2013 at 9:10 PM, Richard Smith <span dir="ltr"><<a href="mailto:richard@metafoo.co.uk" target="_blank">richard@metafoo.co.uk</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="im">On Mon, Jan 21, 2013 at 8:12 PM, Michael Han <<a href="mailto:fragmentshaders@gmail.com">fragmentshaders@gmail.com</a>> wrote:<br>
> Attach updated patch with more tests added; also updated all attributes to<br>
> include spelling index when their ast nodes are constructed.<br>
> OK to commit?<br>
<br>
</div><div class="im">+    OS <<<br>
+      "  case(" << I << ") : {\n"<br>
<br>
</div>Remove the parens here.<br>
<br>
+  // end of the switch statement.<br>
+  OS << "}\n";<br>
+  // end of the print function.<br>
<br>
Please start these comments with a capital letter.<br>
<br>
Other than those tiny things, the patch looks good, but the test<br>
changes seem to be missing from the diff?<br></blockquote><div><br>Yes the tests were missing from the diff, sorry about that. Attach the test patch.<br><br>Michael<br><br> <br></div></div>