Subject: bug#14734: 24.3.50; REGRESSION: defadvice broken wrt doc strings (C-h f)

From: Michael Heerdegen <[email-masked]>
Date: Sat, 29 Jun 2013 00:27:21 +0200

Stefan Monnier <[email-masked]> 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.