Deprecate OffsetView constructors taking index_list_type

[masterleinad]

While looking at #4805, I found a OffsetView constructor taking index_list_type,i.e., std::initializer_list, objects as argument to denote begin and end for every dimension. This pull request deprecates that behavior to use std::pair instead so that specifying too many arguments is not possible.

[dalg24]

Looks fine beside the adding Kokkos::Experimental::range_type

[masterleinad]

Looks fine beside the adding Kokkos::Experimental::range_type

Do you prefer to use std::pair directly instead of the alias then?

[dalg24]

As long we are not injecting an alias in the namespace.
Note that you could write your constructor to take anything that supports std::get<{0,1}>(R const&) with std::tuple_size<R>::value==2

[masterleinad]

Note that you could write your constructor to take anything that supports std::get<{0,1}>(R const&) with std::tuple_size<R>::value==2

I prefer to have a constructor that allows me to do

offset_view_type("o1", {-1, 3}, {-2,2});

and I don't see a need to introduce another templated one taking other tuple-like containers yet.

[dalg24]

Comment about the design choices above the constructor.

[masterleinad]

Retest this please.