[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
bug#14734: 24.3.50; REGRESSION: defadvice broken wrt doc strings (C-h f)
From: |
Michael Heerdegen |
Subject: |
bug#14734: 24.3.50; REGRESSION: defadvice broken wrt doc strings (C-h f) |
Date: |
Sat, 29 Jun 2013 00:27:21 +0200 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/24.3.50 (gnu/linux) |
Stefan Monnier <monnier@iro.umontreal.ca> writes:
> > :around advice: `ad-Advice-isearch-forward'
> [...]
> > It is also incorrect. The advice is :before, not :around.
>
> It is correct: the ad-Advice-isearch-forward advice is an around advice
> (and it is not your advice, it's internal as you say).
> If you click on it, you'll see your before advice, properly labeled.
>
> If you don't want to see the internal thingy, then use the new
> advice-add rather than the old defadvice.
Let me add that advice.el is not yet obsolete, and the manual still
explains the old advice.el. We should try reach a consistent state,
i.e. C-h f should report information according to the package the user
is told to use.
Michael.
bug#14734: 24.3.50; REGRESSION: defadvice broken wrt doc strings (C-h f),
Michael Heerdegen <=
- bug#14734: 24.3.50; REGRESSION: defadvice broken wrt doc strings (C-h f), Stefan Monnier, 2013/06/28
- bug#14734: 24.3.50; REGRESSION: defadvice broken wrt doc strings (C-h f), Drew Adams, 2013/06/28
- bug#14734: 24.3.50; REGRESSION: defadvice broken wrt doc strings (C-h f), Stefan Monnier, 2013/06/28
- bug#14734: 24.3.50; REGRESSION: defadvice broken wrt doc strings (C-h f), Drew Adams, 2013/06/28
- bug#14734: 24.3.50; REGRESSION: defadvice broken wrt doc strings (C-h f), Stefan Monnier, 2013/06/28
- bug#14734: 24.3.50; REGRESSION: defadvice broken wrt doc strings (C-h f), Juanma Barranquero, 2013/06/29
- bug#14734: 24.3.50; REGRESSION: defadvice broken wrt doc strings (C-h f), Stefan Monnier, 2013/06/29