Skip to content

Parse directly to numbers#25493

Open
planetis-m wants to merge 3 commits intonim-lang:develfrom
planetis-m:patch-36
Open

Parse directly to numbers#25493
planetis-m wants to merge 3 commits intonim-lang:develfrom
planetis-m:patch-36

Conversation

@planetis-m
Copy link
Contributor

No description provided.

@planetis-m
Copy link
Contributor Author

Error: unhandled exception: Incorrect JSON kind. Wanted '{JInt}' in '' but got 'JString'. [JsonKindError]

In std/jsontutils.nim#225

@planetis-m
Copy link
Contributor Author

for packedjson a parseNumbers = true parameter in eat and getTok can be introduced which delegates to parseNumber and guards parseNumberValue.

@planetis-m
Copy link
Contributor Author

I suspect that nimbledata2.json in the CI had errors but with this patch it fails at:

raiseParseErr(p, "{")
instead of raising ValueError. CI pass will prove that.

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