Committed in r173358. Thanks for the review!<br><br>Michael<br><br><div class="gmail_quote">On Wed, Jan 23, 2013 at 9:55 PM, Michael Han <span dir="ltr"><<a href="mailto:fragmentshaders@gmail.com" target="_blank">fragmentshaders@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><br><div class="gmail_quote"><div class="im">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>On Mon, Jan 21, 2013 at 8:12 PM, Michael Han <<a href="mailto:fragmentshaders@gmail.com" target="_blank">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>+    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><div><br>Yes the tests were missing from the diff, sorry about that. Attach the test patch.<span class="HOEnZb"><font color="#888888"><br><br>Michael<br><br>
 <br></font></span></div></div>
</blockquote></div><br>