Rename pointer data size type contiguous

[crtrott]

No description provided.

[mhoemmen]

Suggested change

	// Corentins demo compiler for subscript chokes on empty [] call,
	// Early implementations of P2128 (multiple-parameter operator[]) fail to compile
	// if called with zero arguments, even though the proposal supports this.

[mhoemmen]

Suggested change

// though I believe the proposal supports it?

[mhoemmen]

nvc++'s build works if I rebase this PR atop my PR #148. I get one build warning (which explains the one loop change in PR #148):

[ 92%] Building CXX object tests/CMakeFiles/test_mdarray_ctors.dir/test_mdarray_ctors.cpp.o
".../include/experimental/__p0009_bits/layout_right.hpp", line 174: warning: pointless comparison of unsigned integer with zero
        for(rank_type r=0; r<extents_type::rank(); r++) value*=__extents.extent(r);
                            ^
          detected during:
            instantiation of "std::experimental::layout_right::mapping<Extents>::index_type std::experimental::layout_right::mapping<Extents>::required_span_size() const noexcept [with Extents=std::experimental::extents<unsigned int>]" at line 237 of ".../include/experimental/__p1684_bits/mdarray.hpp"
            instantiation of "std::experimental::mdarray<ElementType, Extents, LayoutPolicy, Container>::mdarray(const std::experimental::mdarray<ElementType, Extents, LayoutPolicy, Container>::extents_type &, const Alloc &) [with ElementType=int, Extents=std::experimental::extents<unsigned int>, LayoutPolicy=std::experimental::layout_right, Container=std::vector<int, std::allocator<int>>, Alloc=std::allocator<int>, <unnamed>=0]" at line 365 of ".../tests/test_mdarray_ctors.cpp"

[mhoemmen]

I'm not sure why github is more conservative about conflicts than git. Rebasing #147 atop #149 worked for me.

[crtrott]

Because it requires a force push due to rewriting history I guess.

[mhoemmen]

Thanks @crtrott ! : - D It builds for me with nvc++ and passes all the tests.