[Webkit-unassigned] [Bug 32075] Add workaround for json parsed as unicode

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=32075


Eric Seidel <eric at webkit.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
  Attachment #44172|review?                     |review-
               Flag|                            |




--- Comment #6 from Eric Seidel <eric at webkit.org>  2009-12-11 17:43:13 PST ---
(From update of attachment 44172)
OK.  I think in general this looks fine.

Typo:
 323     Some simplejson version will always return unicode strings. Python

"versions"

Do we know which versions?  It would be best if the comment documented which
versions were bad so that we would have some idea when we could rip out this
hack.  We could even make the hack a noop for newer versions, although that
seems unecesary optimization.

I'm not sure that your isinstance checks are the slickest way to do this via
python, but I don't know how else you'd differentiate between a dict and a
sequence.

In the end Mark Rowe will have to push this new version to the master server,
so he'll see this code go by anyway.

I'm happy to r+ a version with the fixed comment (ideally one which includes
version information).  Feel free to ping me via email if I don't see your
review go by right away.

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.