[libcxx-commits] [PATCH] D123466: [libcxx][ranges] add views::join adaptor object. added test coverage to join_view

Hui via Phabricator via libcxx-commits libcxx-commits at lists.llvm.org
Wed Apr 13 02:09:00 PDT 2022


huixie90 added inline comments.


================
Comment at: libcxx/test/std/ranges/range.adaptors/range.join.view/adaptor.pass.cpp:37
+concept CanBePiped = requires(View&& view, T&& t) {
+                       { std::forward<View>(view) | std::forward<T>(t) };
+                     };
----------------
var-const wrote:
> Nit: I think concepts are usually indented 2 (sometimes 4) spaces in our code base, I'd move this line way left.
lol. I adopted your suggestion but arc linter put the indentation back. what is the command to disable arc linter?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D123466



More information about the libcxx-commits mailing list