Add eof check for streaming parser #321

wardlem · 2021-05-07T14:16:02Z

Summary

Adds an end of file check to the streaming parser.

# This works
JSON.parse("123")

# This also works
JSON.parse(IOBuffer("123\n"))

# This currently breaks
JSON.parse(IOBuffer("123"))

Fixes #314

KristofferC · 2021-05-07T14:30:16Z

test/regression/issue314.jl

@@ -0,0 +1,2 @@
+@test JSON.parse(IOBuffer("123")) == 123


No need to create a new file for this, just add it to e.g.

JSON.jl/test/runtests.jl

Line 36 in 1c24980

@testset "Miscellaneous" begin

Makes sense. I will change this.

pitipatw · 2023-04-20T07:33:09Z

Is this issue resolved?
I also got an error that seems to be related to this one:

Unexpected end of input
│ ...when parsing byte with value '0'

Add eof check for streaming parser

968b49a

wardlem mentioned this pull request May 7, 2021

Unexpected end of input error when parsing input streams #314

Open

KristofferC reviewed May 7, 2021

View reviewed changes

Move stream eof tests to miscellaneous section

d88060e

wardlem requested a review from KristofferC May 13, 2021 11:28