Add a couple of array types #14
Open
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Here is some code I used to parse arrays. It is not complete and it is a good candidate to be moved to C when it is stable enough. All array types are not covered and could require further adjustments.
It has a bug that a value in the array with comma like this one
{"A, B"}will fail to be parsed. In order to fix it, it requires a proper parser. It returnsnilin case ofscan-numberand raises an exception in other cases, so technically there's no mis-parsing.See for yourself if you'd like to accept this code given this downside.