[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