Merge pull request #229 from N5N3/1stindex

fix for `firstindex(128:static(-1):1)`

Author: chriselrod

src/ranges.jl

@@ -214,7 +214,6 @@ known_step(::Type{<:OptionallyStaticStepRange{<:Any,StaticInt{S}}}) where {S} =
 known_last(::Type{<:OptionallyStaticUnitRange{<:Any,StaticInt{L}}}) where {L} = L::Int
 known_last(::Type{<:OptionallyStaticStepRange{<:Any,<:Any,StaticInt{L}}}) where {L} = L::Int
 
-Base.firstindex(x::OptionallyStaticRange) = first(x)
 @inline function Base.first(r::OptionallyStaticRange)::Int
     if known_first(r) === nothing
         return getfield(r, :start)

test/ranges.jl

@@ -113,5 +113,7 @@
 
     @test @inferred(eachindex(static(-7):static(7))) === static(1):static(15)
     @test @inferred((static(-7):static(7))[first(eachindex(static(-7):static(7)))]) == -7
+
+    @test @inferred(firstindex(128:static(-1):1)) == 1
 end