bug-gnu-emacs
[Top][All Lists]
Advanced

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

bug#66928: [PATCH 2/2] Update names to match the docstring


From: Jeremy Bryant
Subject: bug#66928: [PATCH 2/2] Update names to match the docstring
Date: Sat, 04 Nov 2023 23:34:00 +0000

Attached is a proposed patch to install, with the two changes requested.

This is cosmetic patch.

Attachment: 0002-lisp-emacs-lisp-cl-extra.el-cl-remprop-Update-names-.patch
Description: Text Data

Stefan Monnier <monnier@iro.umontreal.ca> writes:

>> I noticed that for the function below the docstring doesn't match the
>> arg names.
>
> I agree it's an odd choice (requiring the use of `\n(fn SYMBOL PROPNAME)`),
> but it's not a big problem since the (fn SYMBOL PROPNAME) makes sure
> that the right names are presented in `C-h o`.
>
> If you want to install a patch like you suggest, then please remove the
> (fn SYMBOL PROPNAME) since it makes it redundant.
> It'd be considered as a cosmetic patch.
>
> One more thing: the first line of the commit should be a bit more
> specific, e.g. it could start with "(cl-remprop):"
>
>
>         Stefan


reply via email to

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