[cfe-commits] [PATCH] Small clean-up

Chandler Carruth chandlerc at google.com
Thu Jul 7 21:34:03 PDT 2011


On Thu, Jul 7, 2011 at 3:23 PM, John Freeman <jfreeman at cse.tamu.edu> wrote:

> 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.


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).

Thanks for the patch!
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20110707/58bf4e81/attachment.html>


More information about the cfe-commits mailing list