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

Previous Next

Package: emacs;

Reported by: Stefan Kangas <stefankangas <at> gmail.com>

Date: Mon, 10 Feb 2025 14:39:02 UTC

Severity: normal

Found in version 31.0.50

Done: Stefan Kangas <stefankangas <at> gmail.com>

To reply to this bug, email your comments to 76179 AT debbugs.gnu.org.
There is no need to reopen the bug first.

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

View this report as an mbox folder, status mbox, maintainer mbox

Report forwarded to monnier <at> iro.umontreal.ca, bug-gnu-emacs <at> gnu.org:
bug#76179; Package emacs. (Mon, 10 Feb 2025 14:39:02 GMT) Full text and rfc822 format available.
Acknowledgement sent to Stefan Kangas <stefankangas <at> gmail.com>:
New bug report received and forwarded. Copy sent to monnier <at> iro.umontreal.ca, bug-gnu-emacs <at> gnu.org. (Mon, 10 Feb 2025 14:39:02 GMT) Full text and rfc822 format available.

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))"
Information forwarded to bug-gnu-emacs <at> gnu.org:
bug#76179; Package emacs. (Mon, 10 Feb 2025 20:38:02 GMT) Full text and rfc822 format available.

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)))
Reply sent to Stefan Kangas <stefankangas <at> gmail.com>:
You have taken responsibility. (Mon, 10 Feb 2025 21:30:03 GMT) Full text and rfc822 format available.
Notification sent to Stefan Kangas <stefankangas <at> gmail.com>:
bug acknowledged by developer. (Mon, 10 Feb 2025 21:30:03 GMT) Full text and rfc822 format available.

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.
This bug report was last modified 10 days ago.

Previous Next

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