Subject: bug#60758: closed (Re: bug#60758: 29.0.60; while-let uses if-let* convention in contradiction to the docstring)

From: [email-masked] (GNU bug Tracking System)
Date: Fri, 13 Jan 2023 00:38:02 +0000

Your bug report

#60758: 29.0.60; while-let uses if-let* convention in contradiction to the docstring

which was filed against the emacs package, has been closed.

The explanation is attached below, along with your original report.
If you require more details, please reply to [email-masked].

-- 
60758: https://debbugs.gnu.org/cgi/bugreport.cgi?bug=60758
GNU Bug Tracking System
Contact [email-masked] with problems

Subject: Re: bug#60758: 29.0.60; while-let uses if-let* convention in contradiction to the docstring
From: Sean Whitton <[email-masked]>
To: Daniel Mendler <[email-masked]>
Cc: [email-masked]
Date: Thu, 12 Jan 2023 17:37:35 -0700
List-Id: <debbugs-submit.debbugs.gnu.org>

Hello,

On Thu 12 Jan 2023 at 08:21PM +01, Daniel Mendler wrote:

> The docstring of `while-let' says:
>
>      The variable list SPEC is the same as in if-let.
>
> However in the definition `if-let*' is used. `if-let*' differs from
> `if-let' since it doesn't support the special single binding form.
>
>      (while-let (var exp)
>          ...)
>
> Therefore `if-let*' should be replaced with `if-let' in the definition
> of `while-let', bringing the implementation in accordance with the
> docstring. The name `while-let' (in contrast to `while-let*') also
> suggest that the `if-let' spec is supported.

I agree that if-let is right, for these reasons.  Now fixed, thank you.

-- 
Sean Whitton

Subject: 29.0.60; while-let uses if-let* convention in contradiction to the docstring
From: Daniel Mendler <[email-masked]>
To: [email-masked]
Date: Thu, 12 Jan 2023 20:21:57 +0100
List-Id: <debbugs-submit.debbugs.gnu.org>

The docstring of `while-let' says:

     The variable list SPEC is the same as in if-let.

However in the definition `if-let*' is used. `if-let*' differs from
`if-let' since it doesn't support the special single binding form.

     (while-let (var exp)
         ...)

Therefore `if-let*' should be replaced with `if-let' in the definition
of `while-let', bringing the implementation in accordance with the
docstring. The name `while-let' (in contrast to `while-let*') also
suggest that the `if-let' spec is supported.

In GNU Emacs 29.0.60 (build 1, x86_64-pc-linux-gnu, X toolkit, cairo
 version 1.16.0, Xaw scroll bars) of 2022-12-30 built on projects
Repository revision: d086cd6cf877c6ca7af6712f9b79b52dd0caa934
Repository branch: emacs-29
Windowing system distributor 'The X.Org Foundation', version 11.0.12011000
System Description: Debian GNU/Linux 11 (bullseye)