Fix type inference issue in basket access (#243)

* Fixes a type inference issue for basket readout

Closes #242

* Bump version number

Author: tamasgal

Project.toml

@@ -1,7 +1,7 @@
 name = "UnROOT"
 uuid = "3cd96dde-e98d-4713-81e9-a4a1b0235ce9"
 authors = ["Tamas Gal", "Jerry Ling", "Johannes Schumann", "Nick Amin"]
-version = "0.10.7"
+version = "0.10.8"
 
 [deps]
 AbstractTrees = "1520ce14-60c1-5f80-bbc7-55ef81b5835c"

src/iteration.jl

@@ -187,19 +187,23 @@ end
 
 function _localindex_newbasket!(ba::LazyBranch{T,J,B}, idx::Integer, tid::Int) where {T,J,B}
     seek_idx = findfirst(x -> x > (idx - 1), ba.fEntry) #support 1.0 syntax
-    if isnothing(seek_idx)  # no basket found, checking in recovered basket
-        ba.buffer[tid] = basketarray(ba.f, ba.b, -1)  # -1 indicating recovered basket mechanics
-        # FIXME: this range is probably wrong for jagged data with non-empty offsets
-        br = ba.b.fBasketEntry[end] + 1:ba.b.fEntries
-    else
-        seek_idx -= 1
-        ba.buffer[tid] = basketarray(ba.f, ba.b, seek_idx)
-        br = (ba.fEntry[seek_idx] + 1):(ba.fEntry[seek_idx + 1])
-    end
+    br = _get_buffer_range(ba, tid, seek_idx)
     ba.buffer_range[tid] = br
     return idx - br.start + 1
 end
 
+function _get_buffer_range(ba::LazyBranch{T, J, B}, tid::Integer, seek_idx::Integer) where {T,J,B}
+    seek_idx -= 1
+    ba.buffer[tid] = basketarray(ba.f, ba.b, seek_idx)
+    (ba.fEntry[seek_idx] + 1)::Int:(ba.fEntry[seek_idx + 1])::Int
+end
+
+function _get_buffer_range(ba::LazyBranch{T, J, B}, tid::Integer, ::Nothing) where {T,J,B}
+    ba.buffer[tid] = basketarray(ba.f, ba.b, -1)  # -1 indicating recovered basket mechanics
+    # FIXME: this range is probably wrong for jagged data with non-empty offsets
+    (ba.b.fBasketEntry[end] + 1)::Int:ba.b.fEntries::Int
+end
+
 Base.IndexStyle(::Type{<:LazyBranch}) = IndexLinear()
 
 function Base.iterate(ba::LazyBranch{T,J,B}, idx=1) where {T,J,B}