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