Review Request: kjs: Implement JSON.parse

Bernd Buschinski b.buschinski at googlemail.com
Sun May 27 16:30:09 BST 2012


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/105056/
-----------------------------------------------------------

(Updated May 27, 2012, 3:30 p.m.)


Review request for kdelibs.


Changes
-------

- fixed stupid typo, that caused "123.456" to cause an syntax error
- use putDirect instead of put to store properties, we need to by-pass __proto__

now passes all ecma JSON-parse tests

this means removing all 15.12.0 - 15.12.2 tests from broken file (not yet included in diff)


Description
-------

kjs: Implement JSON.parse


Diffs (updated)
-----

  kjs/CMakeLists.txt 1188064 
  kjs/interpreter.cpp cf1acf1 
  kjs/json_object.h PRE-CREATION 
  kjs/json_object.cpp PRE-CREATION 
  kjs/jsonlexer.h PRE-CREATION 
  kjs/jsonlexer.cpp PRE-CREATION 
  kjs/libkjs.map e9f679f 

Diff: http://git.reviewboard.kde.org/r/105056/diff/


Testing
-------


Thanks,

Bernd Buschinski

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20120527/9e41500b/attachment.htm>


More information about the kde-core-devel mailing list