[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
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.
More information about the cfe-commits