Review: Needs Fixing

http://json.org/ says that not only '"' and '\' need to be escaped with a 
backslash but also a solidus '/'. 

And, some control characters have special backspace escapes: backspace (x8 -> 
\b), formfeed (xC -> \f), newline (xA -> \n), carriage return (xD -> \r) and 
HTab (x9 -> \t). Currently, they are serialized as hex.

I would propose to add tests for those as well.
-- 
https://code.launchpad.net/~paul-lucas/zorba/bug-1025622/+merge/115636
Your team Zorba Coders is subscribed to branch lp:zorba.

-- 
Mailing list: https://launchpad.net/~zorba-coders
Post to     : zorba-coders@lists.launchpad.net
Unsubscribe : https://launchpad.net/~zorba-coders
More help   : https://help.launchpad.net/ListHelp

Reply via email to