emacs-devel
[Top][All Lists]
Advanced

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

Re: Proposed changes to text-mode.el


From: Stefan Monnier
Subject: Re: Proposed changes to text-mode.el
Date: Tue, 26 Mar 2019 09:00:34 -0400
User-agent: Gnus/5.13 (Gnus v5.13) Emacs/27.0.50 (gnu/linux)

> 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?

LGTM,


        Stefan




reply via email to

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