[libcxx-commits] [PATCH] D125610: [NFC][libc++][format] Improves naming.

[Louis Dionne via Phabricator via libcxx-commits]

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