[libcxx-commits] [PATCH] D125610: [NFC][libc++][format] Improves naming.
Louis Dionne via Phabricator via libcxx-commits
libcxx-commits at lists.llvm.org
Tue May 31 09:14:25 PDT 2022
ldionne accepted this revision.
ldionne added inline comments.
This revision is now accepted and ready to land.
================
Comment at: libcxx/include/__format/buffer.h:105
-private:
static constexpr size_t __buffer_size_ = 256 / sizeof(_CharT);
+
----------------
Should we expose a public `static size_t __buffer_size()` method instead? Otherwise we could also drop the trailing underscore from `__buffer_size_`, which makes it look like a data member, and use `__buffer_size` directly. No strong opinion.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D125610/new/
https://reviews.llvm.org/D125610
More information about the libcxx-commits
mailing list