[webkit-dev] Modelines

Maciej Stachowiak mjs at apple.com
Thu Aug 24 14:15:37 PDT 2006


On Aug 24, 2006, at 11:55 AM, Leo Savernik wrote:

> Am Donnerstag, 24. August 2006 19:26 schrieb Maciej Stachowiak:
>> On Aug 24, 2006, at 8:44 AM, George Staikos wrote:
>>> It was brought to my attention that patches containing editor
>>> modelines are
>>> being rejected until the modelines are removed.  Modelines are a
>>> great way to
>>> instruct the editor to use the proper coding convention on a given
>>> file,
>>> especially when one works on various projects each with different
>>> coding
>>> conventions.  Is this really such a horrible thing?
>>
>> I put this at the top of all C++ files and headers:
>>
>> // -*- mode: c++; c-basic-offset: 4 -*-
>>
>> I think if we want multiple modelines we should agree on which ones
>> and use them consistently.
>>
>
> How about committing proper .kateconfig, .emacs-dirvars, .viwhatever
> configuration files at the project root instead?

That sounds like a better solution if it works. One tricky aspect: my  
modeline tells emacs to treat a header as C++, but we also have non-C+ 
+ headers; I'm not sure if treating as C++ for indendation purposes  
would hurt or not.

Cheers,
Maciej




More information about the webkit-dev mailing list