[libcxx-commits] [PATCH] D96664: [libc++][format] Implement formatters.

[Arthur O'Dwyer via Phabricator via libcxx-commits]

Quuxplusone added inline comments.


================
Comment at: libcxx/include/format:437-439
+    // Note the code doesn't use std::string::iterator since the unit tests
+    // test with debug iterators and they fail with strings created from
+    // std::to_string.
----------------
Yes! Thank you! Can someone reply to my May 1 email to libcxx-dev about this? :P
I just don't see how debug iterators are intended to ever work in practice.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D96664