Deserialize Inf correctly in some cases (#354)

* Deserialize Inf correctly in some cases

Probably would be better to write Inf, -Inf, NaNs etc as strings so they can always be deserialized correctly.

* Update Test name

* Support Julia 1.0

Author: Zentrik

src/serialization.jl

@@ -55,6 +55,11 @@ function recover(x::Vector)
         else
             xsi = if fn == "evals_set" && !haskey(fields, fn)
                 false
+            elseif fn in ("seconds", "overhead", "time_tolerance", "memory_tolerance") &&
+                fields[fn] === nothing
+                # JSON spec doesn't support Inf
+                # These fields should all be >= 0, so we can ignore -Inf case
+                typemax(ft)
             else
                 convert(ft, fields[fn])
             end

test/SerializationTests.jl

@@ -113,4 +113,15 @@ end
         [BenchmarkTools.Parameters(5.0, 10000, 1, true, 0.0, true, false, 0.05, 0.01)]
 end
 
+@testset "Inf in Paramters struct" begin
+    params = BenchmarkTools.Parameters(Inf, 10000, 1, false, Inf, true, false, Inf, Inf)
+
+    io = IOBuffer()
+    BenchmarkTools.save(io, params)
+    json_string = String(take!(io))
+    json_io = IOBuffer(json_string)
+
+    @test BenchmarkTools.load(json_io) == [params]
+end
+
 end # module