[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
bug#52342: [PATCH] Add Texinfo support for Flymake
From: |
Okamsn |
Subject: |
bug#52342: [PATCH] Add Texinfo support for Flymake |
Date: |
Sun, 19 Dec 2021 16:17:56 +0000 |
On 12/7/21 18:46, João Távora wrote:
> On Tue, Dec 7, 2021 at 11:43 PM João Távora <joaotavora@gmail.com
> <mailto:joaotavora@gmail.com>> wrote:
>
>
> On Tue, Dec 7, 2021 at 11:39 PM Okamsn <okamsn@protonmail.com
> <mailto:okamsn@protonmail.com>> wrote:
>
> On 12/7/21 09:23, João Távora wrote:
> > Not sure about the eval-when-compile for require. It was my
> > expression that require is always "when compile".
> >
> > I've unfortunately been a little away from Emacs lately.
> >
> > João
>
> My understanding is that `eval-when-compile` is used to make
> sure that
> macro definitions are available during compilation and that the
> libraries are not loaded when `texinfo.el` is loaded. The
> macros in the
> patch are `rx`, `cl-loop`, and `flymake-log`.
>
>
> require is always `eval-and-compile` (sorry if I misled you earlier).
> It's OK to use `eval-when-compile` when needing `cl-loop` from `cl-lib`
> if _all_ you need form `cl-lib` is indeed macros.
>
> But for flymake, you need the actual runtime things too, right? So
> I think plain require is better and simpler
>
>
> Hmmm, on second thought, it you're planning on relying on
> autoloads for ultimate loading lazyness.... Hmm, maybe
> you're right...
>
> João
Hello Stefan Monnier,
Do you have any thoughts about the lazy loading of Flymake features used
in the patch discussed in this thread (re-attached for convenience)?
Thank you.
v2-0001-Add-basic-Texinfo-support-for-Flymake.patch
Description: Text Data
- bug#52342: [PATCH] Add Texinfo support for Flymake, Okamsn, 2021/12/06
- bug#52342: [PATCH] Add Texinfo support for Flymake, Stefan Kangas, 2021/12/07
- bug#52342: [PATCH] Add Texinfo support for Flymake, João Távora, 2021/12/07
- bug#52342: [PATCH] Add Texinfo support for Flymake, Okamsn, 2021/12/07
- bug#52342: [PATCH] Add Texinfo support for Flymake, João Távora, 2021/12/07
- bug#52342: [PATCH] Add Texinfo support for Flymake, João Távora, 2021/12/07
- bug#52342: [PATCH] Add Texinfo support for Flymake, Okamsn, 2021/12/07
- bug#52342: [PATCH] Add Texinfo support for Flymake, Okamsn, 2021/12/07
- bug#52342: [PATCH] Add Texinfo support for Flymake,
Okamsn <=
- bug#52342: [PATCH] Add Texinfo support for Flymake, Stefan Kangas, 2021/12/07
- bug#52342: [PATCH] Add Texinfo support for Flymake, Okamsn, 2021/12/09
- bug#52342: [PATCH] Add Texinfo support for Flymake, Stefan Kangas, 2021/12/09