reshape(::Array, Val{N}) always returns an Array

[timholy]

We have a convenience method, reshape(A, Val{N}), that reshapes to N dimensions, "collapsing" trailing dimensions if ndims(A) > N. However, one can also use this to "extend" dimensions, adding trailing 1s to the indices. When the array was an Array, we were returning an Array for ndims(A) >= N but not for ndims(A) <= N.

This was only a couple-line change, but I took the opportunity to move a few lines of code around to help clarify the organization.

[andyferris]

I didn't know about this method. I need to be careful with StaticArrays because this is the obvious way to reshape them (with standard dimension sizes).