← Back to team overview

zorba-coders team mailing list archive

Re: [Merge] lp:~paul-lucas/zorba/bug-1025622 into lp:zorba

 

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.


References