[libcxx-commits] [PATCH] D158749: [libc++] Implement LWG3464(istream::gcount() can overflow)

Yurong via Phabricator via libcxx-commits libcxx-commits at lists.llvm.org
Sat Aug 26 03:33:15 PDT 2023


yronglin marked 3 inline comments as done.
yronglin added a comment.

Many thanks for your review! @Mordante



================
Comment at: libcxx/include/istream:219
+private:
+    inline _LIBCPP_INLINE_VISIBILITY
+    void __inc_gcount() {
----------------
Mordante wrote:
> Please use my suggestion, that matches our latest coding standard.
> 
> Nit: I prefer to have private, protected, public in that order, especially since there is already a private section.
Sorry, somehow I have missed this comments.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D158749



More information about the libcxx-commits mailing list