GNU bug report logs - #76179
31.0.50; Error on 'C-h f bindat-type RET'

Previous Next

To add a comment to this bug, you must first unarchive it, by sending
a message to control AT debbugs.gnu.org, with unarchive 76179 in the body.
You can then email your comments to 76179 AT debbugs.gnu.org in the normal way.

Toggle the display of automated, internal messages from the tracker.

Message #5 received at submit <at> debbugs.gnu.org (full text, mbox):

From: Stefan Kangas <stefankangas <at> gmail.com>
To: bug-gnu-emacs <at> gnu.org
Subject: 31.0.50; Error on 'C-h f bindat-type RET'
Date: Mon, 10 Feb 2025 06:38:11 -0800

I get an error on 'C-h f bindat-type RET'.

Simple reproducer:

    emacs -Q -batch -eval "(progn (require 'bindat) \
        (describe-function 'bindat--type))"

Message #8 received at 76179 <at> debbugs.gnu.org (full text, mbox):

From: Stefan Monnier <monnier <at> iro.umontreal.ca>
To: Stefan Kangas <stefankangas <at> gmail.com>
Cc: 76179 <at> debbugs.gnu.org
Subject: Re: bug#76179: 31.0.50; Error on 'C-h f bindat-type RET'
Date: Mon, 10 Feb 2025 15:37:15 -0500

Stefan Kangas [2025-02-10 06:38:11] wrote:
> I get an error on 'C-h f bindat-type RET'.

I don't.

> Simple reproducer:
>
>     emacs -Q -batch -eval "(progn (require 'bindat) \
>         (describe-function 'bindat--type))"

This one (with the double dash) does signal an out of bounds error,
indeed.  I just pushed a trivial fix for it to `emacs-30`.


        Stefan


diff --git a/lisp/help.el b/lisp/help.el
index 5d4c3e61599..1d87c2209c8 100644
--- a/lisp/help.el
+++ b/lisp/help.el
@@ -2381,7 +2381,7 @@ help-function-arglist
             (dolist (arg arglist)
               (unless (and (symbolp arg)
                            (let ((name (symbol-name arg)))
-                             (if (eq (aref name 0) ?&)
+                             (if (and (> (length name) 0) (eq (aref name 0) ?&))
                                  (memq arg '(&rest &optional))
                                (not (string-search "." name)))))
                 (setq valid nil)))

Message #13 received at 76179-done <at> debbugs.gnu.org (full text, mbox):

From: Stefan Kangas <stefankangas <at> gmail.com>
To: Stefan Monnier <monnier <at> iro.umontreal.ca>
Cc: 76179-done <at> debbugs.gnu.org
Subject: Re: bug#76179: 31.0.50; Error on 'C-h f bindat-type RET'
Date: Mon, 10 Feb 2025 13:29:07 -0800

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

>> Simple reproducer:
>>
>>     emacs -Q -batch -eval "(progn (require 'bindat) \
>>         (describe-function 'bindat--type))"
>
> This one (with the double dash) does signal an out of bounds error,
> indeed.  I just pushed a trivial fix for it to `emacs-30`.

Thanks, that did the trick.  Closing.

Previous Next

GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.