modulo operation was expensive isn't it? it can be done without using module operation at all..

you can see his solution... there you will see why

impressive!

well it's an O(n) solution and there is an obvious O(1) solution

that's why it does not deserve its insane 250 upvotes

The values passed to the gen_range method are invalid. You must instead specify a range and it'll work, like so :rng.gen_range(-2..10)

Put tests in tests module

Nice catch!

Fixed missing borrows from sample tests

Placed tests to tests module

It worked without forking, but I didn't notice that the sample tests use vec! instead of &vec!, so now it should be forked.

I did a sneaky edit. Do I fork once again or does it work now?

Language Version is invalid for rust

Try reselecting the version.

Renamed arr to slice

Calling something else than an array arr looks strange.

Changed from &Vec<u32> to &[u32]