[cfe-commits] Patch for review: add a fixit to remove an unused label.

Chris Lattner clattner at apple.com
Wed Jul 27 11:11:19 PDT 2011


On Jul 27, 2011, at 10:57 AM, David Blaikie wrote:

>> Please capitalize local variable names.
> 
> Hmm - I've seen some inconsistency in that regard in the existing code
> & I tried to find something in the LLVM coding conventions (
> http://llvm.org/docs/CodingStandards.html#ll_naming ) but it didn't
> seem to cover variable names at all. Is that the rule, then, variable
> names follow the type name rules (ThingType, ThingVariable,
> thingFunction)?

Yes, that's the style I would prefer.

-Chris




More information about the cfe-commits mailing list