[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
bug#58396: 29.0.50; Optimization failure for add-to-list
From: |
Stefan Monnier |
Subject: |
bug#58396: 29.0.50; Optimization failure for add-to-list |
Date: |
Tue, 26 Sep 2023 18:06:53 -0400 |
User-agent: |
Gnus/5.13 (Gnus v5.13) |
> What about naming it `macroexp-inhibit-compiler-macros´?
Perfect!
> I also wanted to ask whether to extend that approach to the other advice
> installed in `elisp--local-variables´
That would be nice, but...
> but then noticed that `macroexpand´ is actually a built-in.
`elisp--local-variables´ doesn't use that built-in any more, it uses
`macroexpand-1` instead, which is implemented in `macroexp.el` :-)
But the problem is that the advice wraps the call within
a `condition-case` and that's inconvenient to do in a way which
doesn't interfere too much with "normal use" (e.g. doesn't hide/catch
errors when we don't want to, and doesn't impose too much of
a performance cost either).
Then again, maybe I'm worrying about nothing.
Stefan