[libcxx-commits] [PATCH] D114001: [libc++][format] Adds formatter floating-point.

Victor Zverovich via Phabricator via libcxx-commits libcxx-commits at lists.llvm.org
Sat Jan 1 08:01:36 PST 2022


vitaut added inline comments.


================
Comment at: libcxx/include/__format/parser_std_format_spec.h:761
+   */
+  _LIBCPP_HIDE_FROM_ABI constexpr auto __parse(auto& __parse_ctx)
+      -> decltype(__parse_ctx.begin()) {
----------------
The separation of concerns between `parse` and `__parse` is unclear, they even have identical apidoc comments. I think `__parse` better be merged into `parse`.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D114001



More information about the libcxx-commits mailing list