Issue #3 : stringify alternative before better stringify available (avoid crashes on setter #5

Merged
merged 1 commit into from Dec 6, 2012

2 participants

@frantzmiccoli
Collaborator

I'm working on a patch to json.js and json2.js. This should fix the bug on node codein.

@frantzmiccoli frantzmiccoli REFS Silviu-Marian#3 this fixes getter's issues, another commit will …
…be done to json2 to have a more robust of stringify and then JSON2 should be used instead of that dirty patch
6f3dbe4
@frantzmiccoli frantzmiccoli commented on the diff Dec 6, 2012
stringify.js
+ return typeof c === 'string'
+ ? c
+ : '\\u' + ('0000' + a.charCodeAt(0).toString(16)).slice(-4);
+ }) + '"' : '"' + string + '"';
+ }
+
+ function str(key, holder) {
+ var i,
+ k,
+ v,
+ length,
+ mind = gap,
+ partial,
+ value;
+
+ //
@frantzmiccoli
Collaborator

Interesting part starts here

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
@Silviu-Marian Silviu-Marian merged commit 7e9bf09 into Silviu-Marian:master Dec 6, 2012
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment