[all-commits] [llvm/llvm-project] 000940: [libc++] Refactor the tests for [iterator.range] (...

Louis Dionne via All-commits all-commits at lists.llvm.org
Thu Sep 28 06:04:21 PDT 2023


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 000940e2964d27ea2ce29c107198e718f7de3d63
      https://github.com/llvm/llvm-project/commit/000940e2964d27ea2ce29c107198e718f7de3d63
  Author: Louis Dionne <ldionne.2 at gmail.com>
  Date:   2023-09-28 (Thu, 28 Sep 2023)

  Changed paths:
    M libcxx/include/iterator
    A libcxx/test/std/iterators/iterator.range/begin-end.adl.pass.cpp
    A libcxx/test/std/iterators/iterator.range/begin-end.array.pass.cpp
    A libcxx/test/std/iterators/iterator.range/begin-end.container.pass.cpp
    A libcxx/test/std/iterators/iterator.range/begin-end.initializer_list.pass.cpp
    R libcxx/test/std/iterators/iterator.range/begin-end.pass.cpp
    M libcxx/test/std/language.support/support.initlist/support.initlist.range/begin_end.pass.cpp
    M libcxx/utils/data/ignore_format.txt

  Log Message:
  -----------
  [libc++] Refactor the tests for [iterator.range] (#67496)

The tests were a bit of a mess -- the testing coverage wasn't bad but it
was extremely difficult to see what was being tested and where. I split
up the tests to make them easier to audit for completeness and did such
an audit, adding a few missing tests (e.g. the conditional noexcept-ness
of std::cbegin and std::cend). I also audited the synopsis and adjusted
it where it needed to be adjusted.

This patch is in preparation of fixing #67471.




More information about the All-commits mailing list