GNU bug report logs - #66032
[PATCH] Inline advice documentation into advised function's docstring, after all

Previous Next

Full log

Message #20 received at 66032 <at> debbugs.gnu.org (full text, mbox):

From: Stefan Monnier <monnier <at> iro.umontreal.ca>
To: Jens Schmidt <jschmidt4gnu <at> vodafonemail.de>
Cc: 66032 <at> debbugs.gnu.org, drew.adams <at> oracle.com
Subject: Re: bug#66032: [PATCH] Inline advice documentation into advised
 function's docstring, after all
Date: Mon, 18 Sep 2023 18:19:57 -0400

> I have been trying to understand these quirks and previous bugs of
> functions `advice--make-single-doc' and `advice--make-docstring' better,
> also to probably provide some ERT tests, and would like to focus on one
> issue I came across.  Namely, it seems that your work-around:
>
>   ;; Hack attack!  For advices installed before calling
>   ;; Snarf-documentation, the integer offset into the DOC file will not
>   ;; be installed in the "core unadvised function" but in the advice
>   ;; object instead!  So here we try to undo the damage.
>   (when (integerp (aref flist 4))
>     (setq docfun flist))
>
> is no longer efficient.

Indeed, this is not needed any more (since we don't use pre-dump advice
any more) nor is it working (ever since nadvice.el was made to use OClosures).

>   (Snarf-documentation "DOC")
>
> I get a message
>
>   Docstring slot busy for rename-buffer

[ BTW, I consider this to a be long standing bug in
  `Snarf-documentation`.  But since it's very unusual to call this
  function after dumping, it's a low-priority issue.  ]


        Stefan

Previous Next

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