9 Nov
2011
9 Nov
'11
1 p.m.
On 8 November 2011 11:27, Jianshi Huang <jianshi.huang@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