[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
bug#66928: [PATCH 2/2] Update names to match the docstring
From: |
Stefan Monnier |
Subject: |
bug#66928: [PATCH 2/2] Update names to match the docstring |
Date: |
Sat, 04 Nov 2023 12:13:05 -0400 |
User-agent: |
Gnus/5.13 (Gnus v5.13) |
> 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, 2023/11/04
bug#66928: [PATCH 2/2] Update names to match the docstring, Stefan Monnier, 2023/11/12