[libcxx-commits] [PATCH] D153336: [libc++] Fixes thread::id's operator<<.

Jakub Mazurkiewicz via Phabricator via libcxx-commits libcxx-commits at lists.llvm.org
Tue Jul 4 15:14:46 PDT 2023


JMazurkiewicz added inline comments.


================
Comment at: libcxx/include/__thread/thread.h:184
+
+    __os.flags(__default | ((__flags & ios_base::left) ? ios_base::left : ios_base::right));
+    __os << __id.__id_;
----------------
Should we should use C locale while printing? Values from `std::numpunct` (`thousands_sep` and `grouping`) may affect text representation when type of `__id.__id_` is integral.

I've mentioned it in this issue too: https://github.com/llvm/llvm-project/issues/62073.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D153336



More information about the libcxx-commits mailing list