[alexandria-devel] Bug in shuffle (sequences.lisp)

Nikodemus Siivola nikodemus at random-state.net
Wed Nov 9 13:00:35 UTC 2011


On 8 November 2011 11:27, Jianshi Huang <jianshi.huang at gmail.com> wrote:
> (loop for i from (- end 1) downto start      do (rotatef (aref
> sequence i) (aref sequence (random (+ i 1)))))
> should be
>
> (loop for i from start below end      do (rotatef (aref sequence i)
> (aref sequence (+ i (random (- end i))))))
> otherwise the element selected won't be bound by [start, end),
>
> similar bug in the sequence typed version

Fix pushed, thank you!

Cheers,

 -- nikodemus




More information about the alexandria-devel mailing list