GNU bug report logs - #66509
29.1.50; let-alist should support numeric indexing

Previous Next

Package: emacs;

Reported by: Spencer Baugh <sbaugh <at> janestreet.com>

Date: Thu, 12 Oct 2023 22:04:01 UTC

Severity: wishlist

Tags: patch

Found in version 29.1.50

Done: Eli Zaretskii <eliz <at> gnu.org>

Full log


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

From: Eli Zaretskii <eliz <at> gnu.org>
To: Spencer Baugh <sbaugh <at> janestreet.com>
Cc: 66509-done <at> debbugs.gnu.org, stefankangas <at> gmail.com
Subject: Re: bug#66509: 29.1.50; let-alist should support numeric indexing
Date: Sat, 30 Aug 2025 14:00:37 +0300
> From: Spencer Baugh <sbaugh <at> janestreet.com>
> Cc: stefankangas <at> gmail.com,  66509 <at> debbugs.gnu.org
> Date: Fri, 29 Aug 2025 09:19:30 -0400
> 
> Eli Zaretskii <eliz <at> gnu.org> writes:
> 
> >> From: Spencer Baugh <sbaugh <at> janestreet.com>
> >> Cc: Eli Zaretskii <eliz <at> gnu.org>,  66509 <at> debbugs.gnu.org
> >> Date: Thu, 28 Aug 2025 15:05:08 -0400
> >> 
> >> Stefan Kangas <stefankangas <at> gmail.com> writes:
> >> 
> >> > Stefan Kangas <stefankangas <at> gmail.com> writes:
> >> >
> >> >> reopen 66509
> >> >> thanks
> >> >>
> >> >>>> Pushed to master (commit ae4171efdc6).
> >> >>>>
> >> >>>> I'm therefore closing this bug report.
> >> >>
> >> >> I'm reopening this bug report, based on this request from Robert Pluim:
> >> >>
> >> >>> Would it be possible to update the elisp manual as well (and maybe add
> >> >>> an entry to NEWS)?
> >> >>
> >> >> I think that should be done also, so I'm reopening the bug to track that
> >> >> work.  Spencer, do you think you could you please address it?
> >> >>
> >> >> Bonus points for adding tests, as requested by Eli.
> >> >
> >> > Ping!
> >> 
> >> Done in attached patch.
> >
> > Thanks.
> >
> >> ++++
> >> +** 'let-alist' supports indexing into lists
> >
> > Heading lines in NEWS should end in a period.
> 
> Fixed.
> 
> >> +The macro 'let-alist' now interprets symbols containing numbers as list
> >> +indexes.  For example, '.key.0' looks up 'key' in the alist and then
> >
> > "indices", I guess?
> 
> Fixed.
> 
> > Also, I see no patch for the ELisp manual, although the "+++" mark in
> > NEWS seems to imply that the manual was updated.  But the current
> > manual doesn't seem to say anything about keys that include indices,
> > or did I miss something?
> 
> Oops, fixed.

Thanks, now installed on the master branch, and closing the bug.




This bug report was last modified 20 days ago.

Previous Next


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