[PATCH] D27167: [libc++] Support multibyte decimal_point and thousands_sep

2016-12-05 Thread Eric van Gyzen via Phabricator via cfe-commits
vangyzen abandoned this revision. vangyzen added a comment. I'm abandoning this revision because https://reviews.llvm.org/D24218 is clearly superior. https://reviews.llvm.org/D27167 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lis

[PATCH] D27167: [libc++] Support multibyte decimal_point and thousands_sep

2016-12-05 Thread Eric Fiselier via Phabricator via cfe-commits
EricWF added a comment. Huh, I thought I had already fixed this but apparently I forgot to commit the patch. You can find my fix here: https://reviews.llvm.org/D24218 https://reviews.llvm.org/D27167 ___ cfe-commits mailing list cfe-commits@lists.ll

[PATCH] D27167: [libc++] Support multibyte decimal_point and thousands_sep

2016-11-28 Thread Eric van Gyzen via Phabricator via cfe-commits
vangyzen added a comment. A unit test change in https://reviews.llvm.org/D26979 found this and will continue to test it (on some OSes). https://reviews.llvm.org/D27167 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cg

[PATCH] D27167: [libc++] Support multibyte decimal_point and thousands_sep

2016-11-28 Thread Eric van Gyzen via Phabricator via cfe-commits
vangyzen created this revision. vangyzen added a subscriber: cfe-commits. Herald added a subscriber: emaste. Herald added a reviewer: EricWF. numpunct_byname assumed that decimal_point and thousands_sep were ASCII and simply copied the first byte from them. Add support for multibyte strings in th