[PATCH] D26979: Do not hard-code locale data in unit tests: get it from the OS instead

Eric Fiselier via cfe-commits cfe-commits at lists.llvm.org
Tue Nov 22 11:14:06 PST 2016


EricWF accepted this revision.
EricWF added a reviewer: EricWF.
EricWF added a comment.
This revision is now accepted and ready to land.

LGTM minus inline comments.

Thanks for working on this.



================
Comment at: test/std/localization/locale.categories/facet.numpunct/locale.numpunct.byname/thousands_sep.pass.cpp:50
+        }
         std::locale l(LOCALE_en_US_UTF_8);
         {
----------------
I think we should reset the global locale so it's different than the one we pass to `std::locale l(...)`. That way we're testing that we don't accidental use the global locale.


https://reviews.llvm.org/D26979





More information about the cfe-commits mailing list