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

Sebastian Redl sebastian.redl at getdesigned.at
Thu Jul 28 02:16:59 PDT 2011


On 27.07.2011 20:11, Chris Lattner wrote:
> 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.
>
I was under the impression that a naming discussion a few months back 
actually resulted in an agreement to slowly shift towards lowercase 
local variables.

Sebastian



More information about the cfe-commits mailing list