[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.
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
bug#66928: [PATCH 2/2] Update names to match the docstring,
Jeremy Bryant <=
bug#66928: [PATCH 2/2] Update names to match the docstring, Stefan Monnier, 2023/11/12