Skip to content

Comments

Catch if infinity returned by parse_jsconstant is nothing#371

Closed
IanButterworth wants to merge 2 commits intomasterfrom
ib/jet_fix
Closed

Catch if infinity returned by parse_jsconstant is nothing#371
IanButterworth wants to merge 2 commits intomasterfrom
ib/jet_fix

Conversation

@IanButterworth
Copy link
Member

Fixes

  │││││││││┌ parse(str::String) @ JSON.Parser /Users/ian/Documents/GitHub/JSON.jl/src/Parser.jl:443
  ││││││││││┌ parse(str::String; dicttype::Type{Dict{String, Any}}, inttype::Type{Int64}, allownan::Bool, null::Nothing) @ JSON.Parser /Users/ian/Documents/GitHub/JSON.jl/src/Parser.jl:450
  │││││││││││┌ parse_value(pc::JSON.Parser.ParserContext{Dict{String, Any}, Int64, true, nothing}, ps::JSON.Parser.MemoryParserState) @ JSON.Parser /Users/ian/Documents/GitHub/JSON.jl/src/Parser.jl:164
  ││││││││││││┌ parse_number(pc::JSON.Parser.ParserContext{Dict{String, Any}, Int64, true, nothing}, ps::JSON.Parser.MemoryParserState) @ JSON.Parser /Users/ian/Documents/GitHub/JSON.jl/src/specialized.jl:148
  │││││││││││││ no matching method found `-(::Nothing)` (1/3 union split): JSON.Parser.:-(infinity::Union{Nothing, Bool, Float64})
  ││││││││││││└────────────────────
  ││││││││││││┌ parse_number(pc::JSON.Parser.ParserContext{Dict{String, Any}, Int64, true, nothing}, ps::JSON.Parser.MemoryParserState) @ JSON.Parser /Users/ian/Documents/GitHub/JSON.jl/src/specialized.jl:148
  │││││││││││││ no matching method found `-(::Nothing)` (1/3 union split): JSON.Parser.:-(infinity::Union{Nothing, Bool, Float64})::Union{Float64, Int64}
  ││││││││││││└────────────────────

@IanButterworth IanButterworth changed the title Avoid false nothing in parse_jsconstant return union Catch if infinity returned by parse_jsconstant is nothing Feb 26, 2025
@quinnj
Copy link
Member

quinnj commented Oct 3, 2025

Should be "fixed" (obsolete) after #374

@quinnj quinnj closed this Oct 3, 2025
@giordano giordano deleted the ib/jet_fix branch October 3, 2025 10:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants