Review Request: kjs: Implement JSON.parse

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

This is an automatically generated e-mail. To reply, visit:

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

Review request for kdelibs.


- 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)


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/ e9f679f 




Bernd Buschinski

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <>

More information about the kde-core-devel mailing list