[LLVMdev] [cfe-dev] Commit message policy?

Jordan Rose jordan_rose at apple.com
Wed Sep 24 16:25:28 PDT 2014


> On Sep 24, 2014, at 16:19, Jonathan Roelofs <jonathan at codesourcery.com> wrote:
> 
> 
> 
> On 9/24/14 5:03 PM, Tim Northover wrote:
>> On 24 September 2014 15:51, Renato Golin <renato.golin at linaro.org> wrote:
>>> The idea, AFAIK, is to have a title with 50 chars, separated by an
>>> empty line and an infinite number of 80-col lines following.
>> 
>> I tend to follow something like that anyway so I think it's a good
>> idea, but if anything I'd prefer a longer title line than the body.
>> Particularly with the prefixes we tend to put in there, 50 chars is
>> hardly anything.
> I tend to look at this like I look at superExtremelyLongFunctionNames:
> if it needs a really long description for the summary, maybe it's better
> to break it into a few smaller patches. My $0.02, anyway... :)

Just as an example, the case I've run into is something like:

[analyzer] Separate RegionStore::getElementRegion and ...getFieldRegion

which, even with the abbreviation, is 72 characters long. I can fit it into 70 with an ampersand, but...

Jordan



More information about the llvm-dev mailing list