[libcxx-commits] [PATCH] D116569: [libc++] [ranges] Add namespace __cpo to ranges::{advance, next, prev}.

Arthur O'Dwyer via Phabricator via libcxx-commits libcxx-commits at lists.llvm.org
Wed Jan 5 13:11:04 PST 2022


Quuxplusone updated this revision to Diff 397686.
Quuxplusone edited the summary of this revision.
Quuxplusone added a comment.

Remove namespace indentation after discussion offline with @var-const. I think this should be uncontroversial now.

As mentioned (now) in the commit message: The `ranges::begin` group still uses indentation in //their// namespaces (e.g. all of `namespace ranges::__begin` is indented 2 spaces). I thought about "fixing" that in this PR too, but decided I don't want to open that can of worms yet. Let's get this landed first.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D116569

Files:
  libcxx/include/__iterator/advance.h
  libcxx/include/__iterator/next.h
  libcxx/include/__iterator/prev.h

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D116569.397686.patch
Type: text/x-patch
Size: 3241 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/libcxx-commits/attachments/20220105/c0393618/attachment-0001.bin>


More information about the libcxx-commits mailing list