Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

Add RubyLane's tcl parser #107

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Add RubyLane's tcl parser #107

wants to merge 1 commit into from

Conversation

snoe925
Copy link

@snoe925 snoe925 commented Jan 17, 2020

The RubyLane tcl parser is based on a internal C parser.
This used to be yajl C based. Therefore, it's time to add it to this test
suite. For a single file parser it does fairly well.

The RubyLane tcl parser is based on a internal C parser.
This used to be yajl. Therefore, it's time to add it to this test
suite.
@cyanogilvie
Copy link

Just found this set of test cases, and I'm incorporating it into the rl_json test suite. Adding rl_json support to the comparison would be welcome too, and I'm very interested to see how it stacks up :)

I know one area of divergence from the strict specs are that it accepts // and /**/ comments (long JSON literals in code are much more useful with them). I may add a -strict flag to reject them as an option.

Of course now that this set of examples are being incorporated into the test process for rl_json it will cease to be a useful measure of comparison...

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