Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

Extra trailing commas #569

Closed
jacobwilliams opened this issue Jun 6, 2024 · 1 comment · Fixed by #570
Closed

Extra trailing commas #569

jacobwilliams opened this issue Jun 6, 2024 · 1 comment · Fixed by #570
Labels
nonstandard JSON

Comments

@jacobwilliams
Copy link
Owner

jacobwilliams commented Jun 6, 2024 •
edited
Loading

I think JSON-Fortran is parsing files that have extraneous trailing commas? E.g., {"a" : [1,2,3,]}. I don't remember if that was by choice or is just inadvertent. Maybe add an option to check for this when parsing (default would probably be to allow it so as not to break backward compatibility). But, technically, it is invalid JSON.
@jacobwilliams
Copy link
Owner Author

Note that this is actually a JSON5 feature (see #398)

jacobwilliams added a commit that referenced this issue Jun 9, 2024
@jacobwilliams
added an optional allow_trailing_comma flag
7097bf0 
can be used to disallow extra trailing commas in arrays and objects (it is true by default and thus allowed)
Fixes #569
@jacobwilliams jacobwilliams mentioned this issue Jun 9, 2024
Merged
# for free to join this conversation on GitHub. Already have an account? # to comment
Assignees
No one assigned
Labels
nonstandard JSON
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

added an optional allow_trailing_comma flag jacobwilliams/json-fortran
1 participant
@jacobwilliams