I'm definitely not seeing this one. shuffle() is pass-by-reference so it requires a variable, and it's even been on line 116 for a while, but I don't see that we've ever passed it anything but a variable.

I realized that, but not until I had checked the revision to see what had fixed it only to see that it was completely unrelated. I just wanted to make sure anyone else happening by didn't have to do the same thing.