7 Nov
2016
7 Nov
'16
12:58 p.m.
Andrey Semashev wrote:
The standard random_shuffle doesn't use rng() % n though, it uses rng(n). rng() % n is non-uniform in general.
True, a more generic implementation would use uniform_int_distribution. It's fine for my particular use though, and maybe it would be fine for someone else's case, too.
For taking care of the existing uses of std::random_shuffle, it would probably be easier to provide a drop-in replacement that works the same way.