Replace Base.floatrange with simplified implementation (#792)

* Replace Base.floatrange with simplified implementation
Base.floatrange has bugs and is not public API anymore
See JuliaLang/julia#45336
Use a simplified implementation instead

* Inline floatrange

Author: metab0t

src/hist.jl

@@ -96,7 +96,9 @@ function histrange(lo::F, hi::F, n::Integer, closed::Symbol=:left) where F
             len += one(F)
         end
     end
-    Base.floatrange(start,step,Int(len),divisor)
+    StepRangeLen(Base.TwicePrecision{Float64}((start, divisor)),
+                 Base.TwicePrecision{Float64}((step, divisor)),
+                 Int(len))
 end
 
 histrange(vs::NTuple{N,AbstractVector},nbins::NTuple{N,Integer},closed::Symbol) where {N} =

test/hist.jl

@@ -107,6 +107,10 @@ end
     @test l <= typemin(Int)
     @test h >= typemax(Int)
 
+    # Issue 616/667
+    @test StatsBase.histrange([1.0 for i in 1:100], 10, :left) == 1.0:1.0:2.0
+    @test StatsBase.histrange([1.05 for i in 1:100], 10, :left) == 1.05:1.0:2.05
+
     @test_throws ArgumentError StatsBase.histrange([1, 10], 0, :left)
     @test_throws ArgumentError StatsBase.histrange([1, 10], -1, :left)
     @test_throws ArgumentError StatsBase.histrange([1.0, 10.0], 0, :left)