<div class="gmail_quote">On Thu, Jul 7, 2011 at 3:23 PM, John Freeman <span dir="ltr"><<a href="mailto:jfreeman@cse.tamu.edu">jfreeman@cse.tamu.edu</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
This patch is pretty small. It just cleans up some cases in a switch in ParseExpr.cpp so that they have more consistent bracketing and behavior.</blockquote><div><br></div><div>Generally looks good. I committed with a few tweaks in r134683. Specifically, I removed the unneeded {}s. Most of the cases I found further up the function that don't declare any variables don't use {}s, so that actually seemed more consistent (despite a couple of adjacent cases declaring variables and having {}s).</div>
<div><br></div><div>Thanks for the patch!</div></div>