Rename map_range_to to map_range_to!

Also adjust order of arguments to more closely mimic map_to2 in
abstractarray.jl, although this still needs to be a separate function
to avoid ambiguity warnings.
Rename map_to2 to map_to! since it modifies the destination array

It looks like the name map_to2 is a historical artifact from a day when
there was a separate map_to.
Add @kmsquire's test for #4453