GNU bug report logs - #47025
28.0.50; [feature/native-comp] pcase is preloaded, but not eln-compiled

Previous Next

Package: emacs;

Reported by: Eli Zaretskii <eliz <at> gnu.org>

Date: Tue, 9 Mar 2021 18:00:02 UTC

Severity: normal

Found in version 28.0.50

Done: Eli Zaretskii <eliz <at> gnu.org>

Bug is archived. No further changes may be made.

Full log


View this message in rfc822 format

From: Andrea Corallo <akrl <at> sdf.org>
To: Eli Zaretskii <eliz <at> gnu.org>
Cc: 47025 <at> debbugs.gnu.org
Subject: bug#47025: 28.0.50; [feature/native-comp] pcase is preloaded, but not eln-compiled
Date: Tue, 09 Mar 2021 20:21:21 +0000
Eli Zaretskii <eliz <at> gnu.org> writes:

> I've noticed that the native-comp branch preloads pcase:
>
>   Loading files (native compiled elisp)...
>   Loading emacs-lisp/macroexp (native compiled elisp)...
>   Loading emacs-lisp/pcase...
>   Loading emacs-lisp/macroexp (native compiled elisp)...
>
> whereas on master this doesn't happen.  Why is that?

Mmhh, I think is because of this from loadup.el:

====
(load "emacs-lisp/macroexp")
(if (byte-code-function-p (symbol-function 'macroexpand-all))
    nil
  ;; Since loaddefs is not yet loaded, macroexp's uses of pcase will simply
  ;; fail until pcase is explicitly loaded.  This also means that we have to
  ;; disable eager macro-expansion while loading pcase.
  (let ((macroexp--pending-eager-loads '(skip))) (load "emacs-lisp/pcase"))
  ;; Re-load macroexp so as to eagerly macro-expand its uses of pcase.
  (let ((max-lisp-eval-depth (* 2 max-lisp-eval-depth)))
    (load "emacs-lisp/macroexp")))
====

> need to preload pcase?

I fear I don't have an answer as ATM I don't uderstand why it should be
different compared to master.

  Andrea




This bug report was last modified 4 years and 155 days ago.

Previous Next


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