From: Noam Postavsky <npostavs@gmail.com>
To: Eli Zaretskii <eliz@gnu.org>
Cc: 40727@debbugs.gnu.org, p.stephani2@gmail.com, tino.calancha@gmail.com
Subject: bug#40727: 27.0.91; 'cl-loop ... across ... and' seems broken
Date: Tue, 05 May 2020 21:16:50 -0400 [thread overview]
Message-ID: <87h7wtn9q5.fsf@gmail.com> (raw)
In-Reply-To: <83ftckw5f4.fsf@gnu.org> (Eli Zaretskii's message of "Fri, 01 May 2020 09:11:27 +0300")
tags 40727 fixed
close 40727 27.1
quit
Eli Zaretskii <eliz@gnu.org> writes:
>> From: Noam Postavsky <npostavs@gmail.com>
>> Date: Thu, 30 Apr 2020 19:40:43 -0400
>> Cc: 40727@debbugs.gnu.org, Tino Calancha <tino.calancha@gmail.com>
>>
>> Eli, is it okay to push both of these to emacs-27? Alternatively,
>> reverting both fixes to Bug#29799 in addition to the Bug#29866 fix
>> works:
>
> I prefer reverting, thanks.
Okay, I pushed those two to master, and reverted on emacs-27.
[1: 79e133da03]: 2020-05-05 21:07:58 -0400
Revert "Refix conditional step clauses in cl-loop"
https://git.savannah.gnu.org/cgit/emacs.git/commit/?id=79e133da034cd2d7cccfc5a6eb7db340f2dc45a8
[2: caf155c463]: 2020-05-05 21:07:58 -0400
Revert "cl-loop: Add missing guard condition"
https://git.savannah.gnu.org/cgit/emacs.git/commit/?id=caf155c4638d4704b2a099657153c9abc115720b
[3: de1b33f5a8]: 2020-05-05 21:07:58 -0400
Revert "cl-loop: Calculate the array length just once"
https://git.savannah.gnu.org/cgit/emacs.git/commit/?id=de1b33f5a8c6ceee9be59285f70370c3cb2efd34
[4: 1e09364d67]: 2020-05-05 21:07:58 -0400
; Mark Bug#29799 tests as failing since we reverted the fix
https://git.savannah.gnu.org/cgit/emacs.git/commit/?id=1e09364d677b0fb57efd18369c55e8c2d0e826f5
----
[5: 33c5dc3201]: 2020-05-05 21:05:26 -0400
Revert "cl-loop: Calculate the array length just once"
https://git.savannah.gnu.org/cgit/emacs.git/commit/?id=33c5dc3201c334703c951360dbc7e83b60db4456
[6: 721dad1829]: 2020-05-05 21:06:07 -0400
Don't increment array index in cl-loop twice (Bug#40727)
https://git.savannah.gnu.org/cgit/emacs.git/commit/?id=721dad1829d9c641acd31201b96a0ddf827f0c89
next prev parent reply other threads:[~2020-05-06 1:16 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-04-20 0:33 bug#40727: 27.0.91; 'cl-loop ... across ... and' seems broken Philipp Stephani
2020-04-30 12:30 ` Noam Postavsky
2020-04-30 23:40 ` Noam Postavsky
2020-05-01 6:11 ` Eli Zaretskii
2020-05-06 1:16 ` Noam Postavsky [this message]
2020-05-09 23:13 ` Philipp Stephani
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
List information: https://www.gnu.org/software/emacs/
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=87h7wtn9q5.fsf@gmail.com \
--to=npostavs@gmail.com \
--cc=40727@debbugs.gnu.org \
--cc=eliz@gnu.org \
--cc=p.stephani2@gmail.com \
--cc=tino.calancha@gmail.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
Code repositories for project(s) associated with this public inbox
https://git.savannah.gnu.org/cgit/emacs.git
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).