[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
bug#36539: elec-pair.elc gets surreptitiously loaded (almost) unconditio
From: |
Alan Mackenzie |
Subject: |
bug#36539: elec-pair.elc gets surreptitiously loaded (almost) unconditionally at start up. |
Date: |
Sun, 7 Jul 2019 18:29:41 +0000 |
User-agent: |
Mutt/1.10.1 (2018-07-13) |
Hello, Noam.
On Sun, Jul 07, 2019 at 12:57:16 -0400, Noam Postavsky wrote:
> tags 36539 + patch
> quit
> Alan Mackenzie <acm@muc.de> writes:
> > A decision should be taken as to whether elec-pair.elc should be loaded
> > at start up, and if so, it should be part of the dump. If not, the
> > `require' form in elisp-mode.el should be replaced by an
> > `eval-after-load' form.
> I think it's clear that elec-pair should not be part of the dump, since
> electric-pair-mode is not turned on by default. Looking at the git log,
> it seems I added the require to avoid some bootstrapping problems with
> Bug#24901, but we can move the electric-pair-text-pairs manipulation to
> a hook instead:
Just as a matter of interest, I commented out the pertinent form in
emacs-lisp-mode, did a make bootstrap, emacs -Q, and elec-pair.elc was
still present in my Emacs. :-(
I don't know why, or how, but it is. With your patch (below), have you
tested whether or not elec-pair.elc hasn't been loaded on starting
Emacs? If it hasn't been loaded, you've found some trick that eludes
me.
> >>From cf6057cb5bf6a44d718349776e370b225bc7079f Mon Sep 17 00:00:00 2001
> From: Noam Postavsky <npostavs@gmail.com>
> Date: Sun, 7 Jul 2019 12:22:37 -0400
> Subject: [PATCH] Don't load elec-pair in elisp-mode (Bug#36539)
>
> * lisp/progmodes/elisp-mode.el (emacs-lisp-set-electric-text-pairs):
> New function.
> (emacs-lisp-mode): Add it to electric-pair-mode-hook.
> ---
> lisp/progmodes/elisp-mode.el | 14 ++++++++------
> 1 file changed, 8 insertions(+), 6 deletions(-)
>
> diff --git a/lisp/progmodes/elisp-mode.el b/lisp/progmodes/elisp-mode.el
> index c86277a309..7c77c34b50 100644
> --- a/lisp/progmodes/elisp-mode.el
> +++ b/lisp/progmodes/elisp-mode.el
> @@ -233,6 +233,12 @@ emacs-lisp-mode-hook
> :type 'hook
> :group 'lisp)
>
> +(defun emacs-lisp-set-electric-text-pairs ()
> + (defvar electric-pair-text-pairs)
> + (setq-local electric-pair-text-pairs
> + (append '((?\` . ?\') (?‘ . ?’)) electric-pair-text-pairs))
> + (remove-hook 'electric-pair-mode-hook #'emacs-lisp-set-electric-text-pairs
> t))
> +
> ;;;###autoload
> (define-derived-mode emacs-lisp-mode prog-mode "Emacs-Lisp"
> "Major mode for editing Lisp code to run in Emacs.
> @@ -245,12 +251,8 @@ emacs-lisp-mode
> (defvar project-vc-external-roots-function)
> (lisp-mode-variables nil nil 'elisp)
> (add-hook 'after-load-functions #'elisp--font-lock-flush-elisp-buffers)
> - (unless noninteractive
> - (require 'elec-pair)
> - (defvar electric-pair-text-pairs)
> - (setq-local electric-pair-text-pairs
> - (append '((?\` . ?\') (?‘ . ?’)) electric-pair-text-pairs))
> - (setq-local electric-quote-string t))
> + (add-hook 'electric-pair-mode-hook #'emacs-lisp-set-electric-text-pairs t)
Shouldn't that be .....-text pairs nil t), to make a local value of the
^^^
hook? The remove-hook (above) seems to be expecting a local value.
> + (setq-local electric-quote-string t)
> (setq imenu-case-fold-search nil)
> (add-function :before-until (local 'eldoc-documentation-function)
> #'elisp-eldoc-documentation-function)
> --
> 2.11.0
>
As a matter of interest, what is that 2.11.0 that I keep seeing at the
bottom of patches?
--
Alan Mackenzie (Nuremberg, Germany).
- bug#36539: elec-pair.elc gets surreptitiously loaded (almost) unconditionally at start up., Alan Mackenzie, 2019/07/07
- bug#36539: elec-pair.elc gets surreptitiously loaded (almost) unconditionally at start up., Noam Postavsky, 2019/07/07
- bug#36539: elec-pair.elc gets surreptitiously loaded (almost) unconditionally at start up.,
Alan Mackenzie <=
- bug#36539: elec-pair.elc gets surreptitiously loaded (almost) unconditionally at start up., Noam Postavsky, 2019/07/07
- bug#36539: elec-pair.elc gets surreptitiously loaded (almost) unconditionally at start up., Alan Mackenzie, 2019/07/07
- bug#36539: elec-pair.elc gets surreptitiously loaded (almost) unconditionally at start up., Eli Zaretskii, 2019/07/07
- bug#36539: elec-pair.elc gets surreptitiously loaded (almost) unconditionally at start up., Eli Zaretskii, 2019/07/13
- bug#36539: elec-pair.elc gets surreptitiously loaded (almost) unconditionally at start up., Noam Postavsky, 2019/07/13
- bug#36539: elec-pair.elc gets surreptitiously loaded (almost) unconditionally at start up., Noam Postavsky, 2019/07/18