[Top][All Lists]

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

Re: Proposed changes to text-mode.el

From: Basil L. Contovounesios
Subject: Re: Proposed changes to text-mode.el
Date: Wed, 27 Mar 2019 03:50:26 +0000
User-agent: Gnus/5.13 (Gnus v5.13) Emacs/27.0.50 (gnu/linux)

Stefan Monnier <address@hidden> writes:

>> I attach three patches for lisp/textmodes/text-mode.el arising from the
>> discussion in bug#34671.
>> The first enables lexical-binding and makes some minor simplifications.
>> The second makes text-mode no longer reset indent-line-function to
>> indent-relative locally, a change (info "(elisp) Example Major Modes")
>> has been promising for many moons.  AFAICT, text-mode works even when
>> indent-line-function is not set to indent-relative (as I'd hope).  I've
>> had a quick look through all the modes derived from text-mode in
>> emacs.git and nothing strikes me as an obvious incompatibility with the
>> proposed change.  Are there any significant reasons to keep the current
>> setting of indent-line-function in text-mode?
> The reason why I kept the `setq-local` when I changed the default to
> `indent-relative` was simply to be conservative.
>> The third obsoletes the variable text-mode-variant, which is unused in
>> both emacs.git and elpa.git, and has been recommending derived-mode-p as
>> a better alternative in its docstring for almost two decades.
>> WDYT?

Thanks for the clarification and review.
I'll wait a few more days for others to comment before pushing.


reply via email to

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