[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
bug#29866: 27.0.50; cl-loop: Calculate the array length just once
From: |
Noam Postavsky |
Subject: |
bug#29866: 27.0.50; cl-loop: Calculate the array length just once |
Date: |
Tue, 05 May 2020 22:21:36 -0400 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/27.0.91 (gnu/linux) |
tags 29866 - patch
tags 29866 + wontfix
quit
Tino Calancha <tino.calancha@gmail.com> writes:
> (let ((temp-vec (make-symbol "--cl-vec--"))
> + (temp-len (make-symbol "--cl-len--"))
> (temp-idx (make-symbol "--cl-idx--")))
> (push (list temp-vec (pop cl--loop-args)) loop-for-bindings)
> + (push (list temp-len `(length ,temp-vec)) loop-for-bindings)
> (push (list temp-idx -1) loop-for-bindings)
I reverted this because it breaks with 'and' because the bindings are
made with 'let' rather than 'let*' (see Bug#40727). I'm marking wontfix
because I don't intend to anything more about it, but I wouldn't oppose
another attempt that didn't have this problem (I think it would require
extensive code changes though).
[Prev in Thread] |
Current Thread |
[Next in Thread] |
- bug#29866: 27.0.50; cl-loop: Calculate the array length just once,
Noam Postavsky <=