Fix some constructors

Author: c42f

src/MArray.jl

@@ -71,7 +71,7 @@ end
     end
 end
 
-@inline MArray(a::StaticArray) = MArray{size_tuple(typeof(a))}(Tuple(a))
+@inline MArray(a::StaticArray) = MArray{size_tuple(Size(a))}(Tuple(a))
 
 # Simplified show for the type
 show(io::IO, ::Type{MArray{S, T, N}}) where {S, T, N} = print(io, "MArray{$S,$T,$N}")

src/SArray.jl

@@ -50,7 +50,7 @@ end
     end
 end
 
-@inline SArray(a::StaticArray) = SArray{size_tuple(a)}(Tuple(a)) # TODO fixme
+@inline SArray(a::StaticArray) = SArray{size_tuple(Size(a))}(Tuple(a))
 
 # Simplified show for the type
 show(io::IO, ::Type{SArray{S, T, N}}) where {S, T, N} = print(io, "SArray{$S,$T,$N}")

src/traits.jl

@@ -86,6 +86,8 @@ Length(::Type{SA}) where {SA <: StaticArray} = Length(Size(SA))
 
 @pure @inline Base.sub2ind(::Size{S}, x::Int...) where {S} = sub2ind(S, x...)
 
+@pure size_tuple(::Size{S}) where {S} = Tuple{S...}
+
 # Some @pure convenience functions for `Length`
 @pure get(::Length{L}) where {L} = L
 

test/MArray.jl

@@ -27,6 +27,11 @@
         @test MArray{Tuple{2,2},Int}((1,2,3,4)).data === (1,2,3,4)
         @test MArray{Tuple{2,2}}((1,2,3,4)).data === (1,2,3,4)
 
+        @test MArray(SVector(1,2)) isa MArray{Tuple{2}}
+        # Constructors should create a copy (#335)
+        v = MArray{Tuple{2}}(1,2)
+        @test MArray(v) !== v && MArray(v) == v
+
         @test ((@MArray [1])::MArray{Tuple{1}}).data === (1,)
         @test ((@MArray [1,2])::MArray{Tuple{2}}).data === (1,2)
         @test ((@MArray Float64[1,2,3])::MArray{Tuple{3}}).data === (1.0, 2.0, 3.0)

test/SArray.jl

@@ -25,6 +25,8 @@
         @test SArray{Tuple{2,2},Int}((1,2,3,4)).data === (1,2,3,4)
         @test SArray{Tuple{2,2}}((1,2,3,4)).data === (1,2,3,4)
 
+        @test SArray(SArray{Tuple{2}}(1,2)) === SArray{Tuple{2}}(1,2)
+
         @test ((@SArray [1])::SArray{Tuple{1}}).data === (1,)
         @test ((@SArray [1,2])::SArray{Tuple{2}}).data === (1,2)
         @test ((@SArray Float64[1,2,3])::SArray{Tuple{3}}).data === (1.0, 2.0, 3.0)