Re: [Boost-bugs] [Boost C++ Libraries] #4326: Property tree json reader and writer aren't standards-compliant

Subject: Re: [Boost-bugs] [Boost C++ Libraries] #4326: Property tree json reader and writer aren't standards-compliant
From: Boost C++ Libraries (noreply_at_[hidden])
Date: 2010-06-09 15:32:00

#4326: Property tree json reader and writer aren't standards-compliant
--------------------------------------------+-------------------------------
 Reporter:  Leon Mergen <leon@…>            |       Owner:  cornedbee    
     Type:  Bugs                            |      Status:  new          
Milestone:  Boost 1.43.0                    |   Component:  property_tree
  Version:  Boost 1.44.0                    |    Severity:  Problem      
 Keywords:                                  |  
--------------------------------------------+-------------------------------

Comment(by Leon Mergen <leon@…>):

 I'm sorry, the patches were backwards:

 {{{
 --- ./json_parser_write.hpp.old 2010-06-09 15:24:51.000000000 +0000
 +++ ./json_parser_write.hpp     2010-06-09 15:15:47.000000000 +0000
 @@ -36,6 +36,7 @@
              else if (*b == Ch('\r')) result += Ch('\\'), result +=
 Ch('r');
              else if (*b == Ch('"')) result += Ch('\\'), result +=
 Ch('"');
              else if (*b == Ch('\\')) result += Ch('\\'), result +=
 Ch('\\');
 +            else if (*b == Ch('/')) result += Ch('\\'), result +=
 Ch('/');
              else
              {
                  if (std::isprint(*b, loc))



 --- ./json_parser_read.hpp.old  2010-06-09 15:24:43.000000000 +0000
 +++ ./json_parser_read.hpp      2010-06-09 15:20:17.000000000 +0000
 @@ -128,6 +128,7 @@
                  {
                      case Ch('\"'): c.string += Ch('\"'); break;
                      case Ch('\\'): c.string += Ch('\\'); break;
 +                    case Ch('/'): c.string += Ch('/'); break;
                      case Ch('0'): c.string += Ch('\0'); break;
                      case Ch('b'): c.string += Ch('\b'); break;
                      case Ch('f'): c.string += Ch('\f'); break;
 @@ -246,7 +247,7 @@
                          ;

                  escape
 -                    =
 chset_p(detail::widen<Ch>("\"\\0bfnrt").c_str())[typename
 Context::a_escape(self.c)]
 +                    =
 chset_p(detail::widen<Ch>("\"\\/0bfnrt").c_str())[typename
 Context::a_escape(self.c)]
                          | 'u' >> uint_parser<unsigned long, 16, 4,
 4>()[typename Context::a_unicode(self.c)]
                          ;

 }}}

-- 
Ticket URL: <https://svn.boost.org/trac/boost/ticket/4326#comment:1>
Boost C++ Libraries <http://www.boost.org/>
Boost provides free peer-reviewed portable C++ source libraries.