[libcxx-commits] [PATCH] D101242: [libcxx][nfc] renames test iterator types to `legacy_*_iterator`

Arthur O'Dwyer via Phabricator via libcxx-commits libcxx-commits at lists.llvm.org
Mon Apr 26 10:04:28 PDT 2021


Quuxplusone added a comment.

> The intention of this patch is to optimise for the reader, not the writer.

As a reader myself, I explicitly state that this patch //does not// optimize for my preferences when reading code. It makes the code actively harder to read.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D101242



More information about the libcxx-commits mailing list