[libcxx-commits] [PATCH] D120799: [libcxx] [test] Fix get_string_en_US, get_long_double_en_US for Windows

Arthur O'Dwyer via Phabricator via libcxx-commits libcxx-commits at lists.llvm.org
Thu Mar 3 10:27:46 PST 2022


Quuxplusone accepted this revision as: Quuxplusone.
Quuxplusone added inline comments.


================
Comment at: libcxx/test/std/localization/locale.categories/category.monetary/locale.money.get/locale.money.get.members/get_long_double_en_US.pass.cpp:64
+}
+
 int main(int, char**)
----------------
Could this go in `LocaleHelpers` instead? Or would that be problematic because the behavior depends on the specific locale too? (Or is that a good thing because it indicates that this function should actually be named `currency_negate_en_US` or something?)

Worth thinking about, but IMHO not worth my blocking this PR over. :)


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D120799



More information about the libcxx-commits mailing list