emacs-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: The current state of the comment-cache branch


From: Nikolaus Rath
Subject: Re: The current state of the comment-cache branch
Date: Wed, 28 Dec 2016 15:58:53 -0800
User-agent: Gnus/5.130014 (Ma Gnus v0.14) Emacs/24.5 (gnu/linux)

On Dec 28 2016, Eli Zaretskii <address@hidden> wrote:
>> From: Nikolaus Rath <address@hidden>
>> Date: Wed, 28 Dec 2016 08:45:15 -0800
>> 
>> >> > I'm still wondering how useful this could be for C code though. There 
>> >> > are
>> >> > many coding conventions where the opening { of a function does not go in
>> >> > the left-most column. As far as I understand, such coding conventions 
>> >> > would
>> >> > lead to slower parsing?
>> >> 
>> >> This convention is not useful for C.
>> >
>> > Our coding standards clearly call for starting a function's body with
>> > an opening brace in column zero.
>> 
>> I would hope that Emacs is used not only to edit its own source code but
>> also other code, for which these standards don't apply.
>
> It is, but how is that relevant to the issue at hand?  The issue at
> hand is how does open-paren-in-column-0-is-defun-start affect C code
> editing.  The intent of my reference to GNU standards was to point
> out that having an opening brace in column zero is a very frequent
> situation in the C sources of any GNU project, and therefore saying
> this convention is not useful for C flies in the face of the reality
> right under our feet.

What I meant is that setting open-paren-in-column-0-is-defun-start to t
in general is not a good idea, because it does not work for all C
files. So for this option to be useful, you'd have to do set it
conditionally - which IMO restricts its usefulness a lot.

Best,
-Nikolaus

-- 
GPG encrypted emails preferred. Key id: 0xD113FCAC3C4E599F
Fingerprint: ED31 791B 2C5C 1613 AF38 8B8A D113 FCAC 3C4E 599F

             »Time flies like an arrow, fruit flies like a Banana.«



reply via email to

[Prev in Thread] Current Thread [Next in Thread]