GNU bug report logs - #55149
29.0.50; Commit f30625943e broke magit/with-editor

Previous Next

Package: emacs;

Reported by: Tassilo Horn <thorn <at> fastmail.fm>

Date: Wed, 27 Apr 2022 12:59:01 UTC

Severity: normal

Tags: patch

Merged with 55160, 55161

Found in version 29.0.50

Done: Stefan Monnier <monnier <at> iro.umontreal.ca>

Bug is archived. No further changes may be made.

Full log


Message #94 received at 55149-done <at> debbugs.gnu.org (full text, mbox):

From: Tassilo Horn <thorn <at> fastmail.fm>
To: Stefan Monnier <monnier <at> iro.umontreal.ca>
Cc: 55149-done <at> debbugs.gnu.org, No Wayman <iarchivedmywholelife <at> gmail.com>,
 Juri Linkov <juri <at> linkov.net>
Subject: Re: bug#55149: 29.0.50; Commit f30625943e broke magit/with-editor
Date: Thu, 28 Apr 2022 16:39:11 +0200
Stefan Monnier <monnier <at> iro.umontreal.ca> writes:

>> It is, thanks.  Shouldn't you also adapt the add-function docstring?
>> Oh, that's adapted automatically!  Interesting approach.
>
> :-)
> Actually, it was done to avoid copy&paste between the docstrings of
> `add-function` and `advice-add`, but it's nice that it also avoids
> duplication between the code and the doc and makes sure the doc is in
> sync with the code.

Indeed, and I think until recently advice-add didn't document HOW but
just referred to add-function which was a bit annoying because usually
one uses advice-add so does C-h f on that and then needs to TAB to
add-function to get the docs on that.

Would you mind adding this magic also to define-advice so that the HOW
is described there as well?

Bye,
Tassilo




This bug report was last modified 3 years and 24 days ago.

Previous Next


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