On 2026-01-26 10:38:06 +0800, Kevin J. McCarthy wrote:
>     Also, I personally dislike tabs, so I have:
>       (setq-default indent-tabs-mode nil)
>     in my own configuration.

I also disable indent-tabs-mode by default (but I have a find-file
hook for projects that use tabs).

IMHO, unless all indented lines use tabs (so, never indentations
with 2 spaces), tabs should never be used as indentation is broken
as soon as a prefix is added (e.g. in diffs and in quoted code).

Moreover, tabs are rendered incorrectly on the web and they are not
always preserved by copy-paste (e.g. never in xterm). So I think
that they should never be used.

> * Commit efa3afb5: Clean up formatting.
>     Add spaces after if, else, while, for, switch.

Yes! (I suppose that this does not apply for "else", though.)

>     Unify the brace placement style.  The vast majority of the code uses
>     Allman style so convert the relatively few K&R braces over.

And unify the function-call style, i.e. whether a space between
the function name and the opening parenthesis is present?

The same applies to function-like macros.

-- 
Vincent Lefèvre <[email protected]> - Web: <https://www.vinc17.net/>
100% accessible validated (X)HTML - Blog: <https://www.vinc17.net/blog/>
Work: CR INRIA - computer arithmetic / Pascaline project (LIP, ENS-Lyon)

Reply via email to