I don't see a thing wrong with your second shuffle. It works for me. Are you sure your testing was correct and applied to the arrays you thought?

There is no reason but, perhaps, economy, that an array reference is used there.

After Compline,
Zaxo