[LLVMdev] DIFactory

[Chris Lattner]

On Feb 18, 2011, at 2:19 PM, Talin wrote:

> 
> Yeah, I remember that conversation - I just didn't know that any action had been taken.
> 
> However, my other concern is this: According to the LLVM Coding Standards document (http://llvm.org/docs/CodingStandards.html#ll_naming):
> 
> Function names
>  should be verb phrases (as they represent actions), and command-like function should be imperative. The name should be camel case, and start with a lower case letter (e.g. openFile() or isFoo()).
> 
> The coding standards say functions should begin with a lower case letter, but I see a lot of new code (not just DIBuilder) that uses method names that begin with an upper case letter. Is the document incorrect, or is there basically no enforcement? 

DIBuilder is older than the naming standards. Patches to improve DIBuilder's naming would definitely be appreciated.

-Chris
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-dev/attachments/20110219/6b7fc5d3/attachment.html>