boostorg / property_tree

Boost.org property_tree module
http://boost.org/libs/property_tree
55 stars 93 forks source link

Fix escape misbehavior on basic_string<char>. #7

Closed berkus closed 9 years ago

berkus commented 10 years ago

Signed entity used for byte comparison, which fails for signed char. Due to this JSON serialization spits \uXXXX monstrosity instead of proper UTF-8.

maxim-ky commented 9 years ago

Nice patch, thanks :) I cannot wait for boost release so I plan to use it just now

berkus commented 9 years ago

Beware that property_tree doesn't use proper unicode deserialization either and may fail.

maxim-ky commented 9 years ago

Thanks for the caution, but deserialization works well with the used unicode subset (unlike serialization)

CornedBee commented 9 years ago

Pulled into develop branch with a fix for MSVC, which really doesn't handle non-ASCII characters in narrow string literals well. Thank you for the patches.