Merge pull request #112 from chriselrod/elsize

Fix and properly test elsize
SciML · Feb 28, 2022 · 9584c99 · 9584c99
2 parents 8de79b0 + 2ace707
commit 9584c99
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 1 deletion.
diff --git a/src/larray.jl b/src/larray.jl
@@ -270,5 +270,5 @@ function Base.vcat(x::LArray, y::LArray)
     LArray{(LabelledArrays.symnames(typeof(x))...,LabelledArrays.symnames(typeof(y))...)}(vcat(x.__x,y.__x))
 end
 
-Base.elsize(::Type{<:LArray{T}}) where {T} = T
+Base.elsize(::Type{<:LArray{T}}) where {T} = sizeof(T)
 
diff --git a/test/larrays.jl b/test/larrays.jl
@@ -4,6 +4,7 @@ using LabelledArrays, Test, InteractiveUtils
     vals = [1.0,2.0,3.0]
     syms = (:a,:b,:c)
     x = @LArray vals syms
+    @test Base.elsize(x) == Base.elsize(vals) == 8
     @test_nowarn display(x)
     y = @LVector Float64 (:a,:b,:c)
     y .= [1,2,3.]