--- Begin Message ---
Subject: |
30.0.50; Small issues with defcustoms in ERC |
Date: |
Fri, 13 Oct 2023 10:33:08 -0300 |
User-agent: |
Mozilla Thunderbird |
Looking for defcustom docstrings and :type inconsistencies, I found the
following in erc/:
The docstring of erc-fill-variable-maximum-indentation says its value
can be nil, but its :type is just integer.
The choices in erc-keep-place-indicator-style look like a copy-pasta
from erc-keep-place-indicator-buffer-type. The docstring matches what
the code checks, so I guess the choices are wrong.
Finally, erc-networks-alist says that MATCHER can be either a regexp or
a function. But neither erc-determine-network (obsoleted) nor
erc-networks--determine check to see if MATCHER is a function. So maybe
that isn't supported?
--- End Message ---
--- Begin Message ---
Subject: |
Re: bug#66520: 30.0.50; Small issues with defcustoms in ERC |
Date: |
Fri, 13 Oct 2023 08:04:48 -0700 |
User-agent: |
Gnus/5.13 (Gnus v5.13) |
Mauro Aranda <maurooaranda@gmail.com> writes:
> tags 66520 patch
> quit
>
>
> I attach a patch to fix these issues.
Appreciate the diligence!
> Maybe I'm wrong about changing the docstring in erc-networks-alist.
Nah, I couldn't find any sign of the function variant being considered
in the 18+ years since the file was first introduced to the old CVS
repo.
I've added this as
https://git.savannah.gnu.org/cgit/emacs.git/commit/?id=baf778c7
Thanks and closing.
--- End Message ---