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

JSON Changes 2 #2

Merged
merged 5 commits into from
Jan 18, 2014
Merged

JSON Changes 2 #2

merged 5 commits into from
Jan 18, 2014

Conversation

fredericaltorres
Copy link
Contributor

I fixed more JSON.stringify() and JSON.parse() tests. only 30 failed test left.

I did change the JavaScriptException to extract the message correctly, so it can be part of the exception.
If you do not like it. Change it with something else and check in.

Fred,

fix 15.12.3-11-2, 15.12.3-11-14, 15.12.3-6-b-3.js, S15.12.2_A1
Fixed some JSON.parse errors and set exception message correctly
…-2-6.js 15.12.1.1-g4-1

fixed unit tests 15.12.1.1-g2-2 15.12.1.1-g2-4 15.12.1.1-g1-2
15.12.2-2-6.js 15.12.1.1-g4-1
@fredericaltorres
Copy link
Contributor Author

All JSON unit tests should pass now

sebastienros added a commit that referenced this pull request Jan 18, 2014
@sebastienros sebastienros merged commit 89ec21a into sebastienros:master Jan 18, 2014
westyside pushed a commit to westyside/jint that referenced this pull request Aug 8, 2016
russlank added a commit to Ellested/jint that referenced this pull request Jul 17, 2020
sebastienros pushed a commit that referenced this pull request Dec 3, 2020
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