emacs-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [External] : Re: [PATCH] When deleting in bookmark menu, prompt for


From: Karl Fogel
Subject: Re: [External] : Re: [PATCH] When deleting in bookmark menu, prompt for confirmation.
Date: Tue, 25 May 2021 15:24:44 -0500
User-agent: Gnus/5.13 (Gnus v5.13) Emacs/28.0.50 (gnu/linux)

On 25 May 2021, Eli Zaretskii wrote:
From: Karl Fogel <kfogel@red-bean.com>
Cc: orontee@gmail.com,  drew.adams@oracle.com,  larsi@gnus.org,
  monnier@iro.umontreal.ca,  emacs-devel@gnu.org
Date: Tue, 25 May 2021 00:38:22 -0500

>> Thoughts?
>
>I'll have thoughts when I see the result ;-)

Heh, fair enough. My inquiry was about the general practice of having one function's doc string referring out to another function's.

It depends on what you need to say, thus my response above.

Given what you wrote, and what bookmark-load does with the prefix
argument, I think it is better to say that explicitly in
bookmark-bmenu-load's doc string, since you only need a single quite simple sentence to say that, whereas the doc strong of bookmark-load
is quite long.

Well, now that I've done it, I think your way is an improvement -- although it turned out to be slightly more doc change than I expected. Revised patch attached for review.

Best regards,
-Karl

Attachment: 0001-Improve-some-doc-strings-in-bookmark.el.patch
Description: Text document


reply via email to

[Prev in Thread] Current Thread [Next in Thread]