Mark Dickinson added the comment:

I see you've added a test for the behaviour of:

    shuffle(seq, random=lambda: -1.0)

I'd suggest leaving that test out: that this works right now is really just an 
accident of the (CPython) implementation, and it may well not work on other 
Python implementations. Adding a test in effect says that there was a 
deliberate decision to support this (which as far as I know there isn't, and 
shouldn't be).

----------

_______________________________________
Python tracker <rep...@bugs.python.org>
<http://bugs.python.org/issue30308>
_______________________________________
_______________________________________________
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

Reply via email to