[libcxx-commits] [PATCH] D118686: [libc++][ranges][NFC] Test the specializations of `tuple_{size, element}` for ranges.

[Arthur O'Dwyer via Phabricator via libcxx-commits]

Quuxplusone added inline comments.


================
Comment at: libcxx/include/ranges:257
 #include <__ranges/size.h>
-#include <__ranges/subrange.h>
+#include <__ranges/subrange.h> // This also makes the specializations of `tuple_{size,element}` available.
 #include <__ranges/take_view.h>
----------------
var-const wrote:
> Quuxplusone wrote:
> > Please remove the comment.
> Why?
Because (1) we don't add comments for any other headers, and (2) the fact that `__ranges/subrange.h` includes the API of `std::ranges::subrange` is obvious. We don't need to say "This makes `subrange` available, and all its methods, and its `get` overload, and its `tuple_size` and `tuple_element`, and its `enable_borrowed_range`, and its `enable_view`, and its..." We don't need to say //any// of that, because it's obvious from the filename.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D118686