ListOffsetArray's is_valid.

JuliaHEP · Aug 8, 2023 · 1891db3 · 1891db3
1 parent 121825a
commit 1891db3
Show file tree

Hide file tree

Showing 2 changed files with 15 additions and 1 deletion.
diff --git a/src/AwkwardArray.jl b/src/AwkwardArray.jl
@@ -63,7 +63,12 @@ struct ListOffsetArray{T<:Union{Int32,UInt32,Int64}} <: Content
 end
 
 function is_valid(x::ListOffsetArray)
-    true
+    for i = Base.firstindex(x):Base.lastindex(x)
+        if x.offsets[i+1] < x.offsets[i]
+            return false
+        end
+    end
+    return true
 end
 
 function Base.length(x::ListOffsetArray)

diff --git a/test/runtests.jl b/test/runtests.jl
@@ -39,4 +39,13 @@ using Test
         end
         @test tmp == 5
     end
+
+    begin
+        layout = AwkwardArray.ListOffsetArray(
+            [0, 3, 2, 5],
+            AwkwardArray.PrimitiveArray([1.1, 2.2, 3.3, 4.4, 5.5]),
+        )
+        @test !AwkwardArray.is_valid(layout)
+    end
+
 end