[libcxx-commits] [PATCH] D59572: Fix and speedup __libcpp_locale_guard on Windows

[Tom Anderson via Phabricator via libcxx-commits]

thomasanderson marked an inline comment as done.
thomasanderson added inline comments.


================
Comment at: include/__locale:85
+      if (__locale_all != nullptr) {
+        __setlocale(__locale_all);
+        free(__locale_all);
----------------
smeenai wrote:
> Have you confirmed that `setlocale` does the right thing when given the semicolon-separated list of locale settings for the different categories? The docs talk about returning such a string for the `setlocale(LC_ALL, nullptr)` case, but they don't mention what happens if you try to use such a string to set the locale. https://docs.microsoft.com/en-us/cpp/c-runtime-library/reference/setlocale-wsetlocale?view=vs-2017
Yes, I've verified that it does have the correct behavior


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D59572/new/

https://reviews.llvm.org/D59572