[libcxx-commits] [PATCH] D65260: [libc++] Consolidate swap, swap_ranges, and iter_swap in <type_traits>. NFC.

Louis Dionne via Phabricator via libcxx-commits libcxx-commits at lists.llvm.org
Thu Jul 25 07:02:13 PDT 2019


ldionne accepted this revision.
ldionne added a comment.
This revision is now accepted and ready to land.

I don't mind this change.

However, I really dislike the general organization (or rather, the lack thereof) in libc++. It's weird to be defining `swap` and `swap_ranges` inside `<type_traits>`, when `swap_ranges` should be part of `<algorithm>` and `swap` should be in `<utility>`.


Repository:
  rCXX libc++

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

https://reviews.llvm.org/D65260





More information about the libcxx-commits mailing list