[libcxx-commits] [PATCH] D103466: [libc++][format] Adds char formatter.

[Victor Zverovich via Phabricator via libcxx-commits]

vitaut added a comment.

LGTM



================
Comment at: libcxx/test/libcxx/utilities/format/format.string/format.string.std/std_format_spec_char.pass.cpp:174-175
+
+  static_assert(std::__format::__number_max == 2'147'483'647,
+                "Update the assert and the test.");
+  // Note the static_assert tests whether the arg-id is valid.
----------------
You already have this assert above.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D103466