GNU bug report logs - #35224
[PATCH] include indentation size in .dir-locals

Previous Next

Full log

Message #22 received at 35224 <at> debbugs.gnu.org (full text, mbox):

From: "Basil L. Contovounesios" <contovob <at> tcd.ie>
To: Konstantin Kharlamov <hi-angel <at> yandex.ru>
Cc: 35224 <at> debbugs.gnu.org
Subject: Re: bug#35224: [PATCH] include indentation size in .dir-locals
Date: Thu, 11 Apr 2019 11:58:37 +0100

Konstantin Kharlamov <hi-angel <at> yandex.ru> writes:

> On Чт, Apr 11, 2019 at 11:26, Basil L. Contovounesios <contovob <at> tcd.ie> wrote:
>
>> Globally setting c-basic-offset overrides the cc-mode style inheritance
>> system.  There are various ways to customise this variable in a more
>> granular way, such as defining a custom style which inherits from a
>> built-in style, or setting c-basic-offset locally in a mode hook.
>> For details, see (info "(ccmode) Config Basics").
>
> Oh, okay, so this works as expected. But since googling "change indentation
> size" always gives using specifically c-basic-offset (e.g.
> https://stackoverflow.com/questions/14939608/how-to-change-emacs-struct-indents-from-4-to-2-spaces),
> I guess it's safe to assume most of Emacs users has it changed. So it's useful
> to add this to dir-locals anyway.

IMO, that's a bug in the configuration of those users because they're
overriding how cc-mode works, and the cc-mode manual describes in detail
the various ways to customise indentation.  So I don't see a need to
change the Emacs dir-locals-file to accommodate this use-case.

But that's just one opinion, and I'm not familiar with prevailing policy
on what to add to the Emacs dir-locals-file.

-- 
Basil

Previous Next

GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.