[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
bug#33301: 27.0.50; broken elisp indentation for non-definition symbols
From: |
Lars Ingebrigtsen |
Subject: |
bug#33301: 27.0.50; broken elisp indentation for non-definition symbols starting with "def.." |
Date: |
Mon, 24 Aug 2020 15:12:34 +0200 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/28.0.50 (gnu/linux) |
João Távora <joaotavora@gmail.com> writes:
>> But the main problem is that it would indent the code differently than
>> it does now, and that leads to whitespace churn in the vc, which we
>> should avoid unless we have a very, very good reason not to.
>
> What exactly do you mean "whitespace churn"? Can you illustrate this
> hypothetical scenario? I don't expect whitespace/indentation beyond
> fixing the akward cases, at least that's the entire point of this
> report.
It means indenting some things in a different way than today? That
leads to whitespace changes.
>>> As for out-of-tree definitions, we could be lenient and have this
>>> saner indentation be controlled by a variable which we would default
>>> to 'insane, but to 'sane inside Emacs's source, via directory local
>>> variables.
>>
>> I'd be against that -- again, because it leads to whitespace VC churn.
>
> Again, I'm missing something: this option wouldn't lead to that, I think
If some people have the variable set to 'insane, they would indent the
code they're writing differently than the rest, which would lead to
whitespace churn.
> PS: another entirely different approach would just limit the current
> hacky heuristic to calls/expansions that happen at top-level, i.e. at
> "column 0". I believe this to be the vast majority (though not the
> entirety) of cases.
That's probably true...
--
(domestic pets only, the antidote for overdose, milk.)
bloggy blog: http://lars.ingebrigtsen.no