[webkit-reviews] review denied: [Bug 20031] Implement ES 3.1 JSON object : [Attachment 30295] Patch to add JSON object - remove tabs

[bugzilla-daemon at webkit.org]

Oliver Hunt <oliver at apple.com> has denied Husam Senussi <husam at senussi.com>'s
request for review:
Bug 20031: Implement ES 3.1 JSON object
https://bugs.webkit.org/show_bug.cgi?id=20031

Attachment 30295: Patch to add JSON object - remove tabs
https://bugs.webkit.org/attachment.cgi?id=30295&action=review

------- Additional Comments from Oliver Hunt <oliver at apple.com>
I don't like this patch, while the JSON object itself is fine, i don't believe
it is necessary to use a yacc parser -- given the simplicity of the JSON
grammar i believe a hand coded recursive descent parser is likely to be faster,
with a little care i believe this will also make it easier to have the parser
avoid ever creating an AST in the case where we do not have custom reviver
functions.

(Addtionally this particular grammar tracks a lot of information that we track
in the normal JS grammar which is of no value to the JSON parser) -- I'm also
not entirely sure i like the lexer, but certainly i prefer it over the grammar
implementation.  We can in fact probably just use the literal parser we
preflight eval with to achieve this.

Sorry for the delay in reviewing.

If you want to discuss more directly i tend to be on #webkit as olliej fairly
continuously.