Details
Description
Shuffle needlessly tests Random.nextInt() for a negative value
It iterates from index N-1 to 0, which means it always swaps index 0 with itself needlessly.
It is only tested with probabilistic tests, the patch adds a proper test. Note that the attached test also passes on the RI.