[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
bug#50869: 28.0.50; tramp: void-function tramp-file-name-method--cmacro
From: |
Lars Ingebrigtsen |
Subject: |
bug#50869: 28.0.50; tramp: void-function tramp-file-name-method--cmacro |
Date: |
Sat, 03 Sep 2022 12:07:43 +0200 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/29.0.50 (gnu/linux) |
Stefan Monnier via "Bug reports for GNU Emacs, the Swiss army knife of
text editors" <bug-gnu-emacs@gnu.org> writes:
> The main point is that putting the `cl-defstruct` into the
> `tramp-loaddefs.el` is really ugly and I think it's asking for trouble.
The cl-defstruct is in there because a couple of defsubsts (that use the
struct) are also in the tramp-loaddefs.el file. So if we remove the
defstruct, we also have to adjust the defsubsts.
> In contrast the OP's "errors" aren't actual errors, they're just
> messages indicating that some optimization could not be applied.
> We should really change the wording to clarify that they're warnings.
>
> (see first hunk in patch below).
Looks good.
> - We could make `cl-defstruct` use `define-symbol-prop` rather than
> `put`, as in the third hunk of the patch below.
Makes sense to me.
- bug#50869: 28.0.50; tramp: void-function tramp-file-name-method--cmacro, Lars Ingebrigtsen, 2022/09/02
- bug#50869: 28.0.50; tramp: void-function tramp-file-name-method--cmacro, Stefan Monnier, 2022/09/02
- bug#50869: 28.0.50; tramp: void-function tramp-file-name-method--cmacro,
Lars Ingebrigtsen <=
- bug#50869: 28.0.50; tramp: void-function tramp-file-name-method--cmacro, Stefan Monnier, 2022/09/03
- bug#50869: 28.0.50; tramp: void-function tramp-file-name-method--cmacro, Michael Albinus, 2022/09/04
- bug#50869: 28.0.50; tramp: void-function tramp-file-name-method--cmacro, Stefan Monnier, 2022/09/04
- bug#50869: 28.0.50; tramp: void-function tramp-file-name-method--cmacro, Michael Albinus, 2022/09/04
- bug#50869: 28.0.50; tramp: void-function tramp-file-name-method--cmacro, Stefan Monnier, 2022/09/04