Spelling error in the "Parsing JSON is a Minefield 💣" article

nst / JSONTestSuite

A comprehensive test suite for RFC 8259 compliant JSON parsers

MIT License

885 stars 142 forks source link

Spelling error in the "Parsing JSON is a Minefield 💣" article #134

Open ChristanBeyond opened 3 days ago

ChristanBeyond commented 3 days ago

The article states:

You'll find more that 300 tests in the JSONTestSuite GitHub repository.

but it should be:

You'll find more than 300 tests in the JSONTestSuite GitHub repository.

RokerHRO commented 3 days ago

There are a lot more spelling errors, even distorting errors, e.g. in section 2.5 Strings, where the examples are wrong due to missing backslashes:

["x00"] is totally valid, I guess the author meant ["\x00"] as an example for invalid string escape.
[""\/bfnrt"] is invalid, because two consecutive quote characters would describe an empty string, no additional characters are allowed. I guess the author wanted to write ["\"\/\b\f\n\r\t"]
["uqqqq"] is valid, but no escape. I guess the author wanted to write ["\uqqqq"]

and many more such things. :-(

ChristanBeyond commented 3 days ago

Would be nice if he gave the article an update again, it's overall a really good article, which helped quite a bit with the paper I'm writing.

RokerHRO commented 3 days ago

ACK, but I don't see much activity here in this repo, so I think the author is not interested in fixing his article or out of time at the moment. :-(

nst commented 3 days ago

Author here! Thank you so much for helping to improve the article. Indeed, escaping errors may be due to Markdown syntax. I'll upload the article to GitHub shortly so that you can submit pull requests.