[libcxx-commits] [PATCH] D142311: [libc++][test] Improves locale validation.

[Alexander Richardson via Phabricator via libcxx-commits]

arichardson added inline comments.


================
Comment at: libcxx/test/std/utilities/format/format.range/format.range.formatter/format.functions.tests.h:590
 #ifndef TEST_HAS_NO_LOCALIZATION
-// TODO FMT Enable with locale testing active
-#  if 0
+#  ifdef TEST_HAS_LOCALE_fr_FR_UTF_8
   std::locale::global(std::locale(LOCALE_fr_FR_UTF_8));
----------------
How about generating a header with these macros set to 0/1, then typos could be caught by -Werror=undef?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D142311