[all-commits] [llvm/llvm-project] 4039a7: [libc++][test] Improve tests for assign in std::ve...

Peng Liu via All-commits all-commits at lists.llvm.org
Thu Dec 19 08:19:48 PST 2024


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 4039a79de71bd969ef5bf944fd9f46430338ff7e
      https://github.com/llvm/llvm-project/commit/4039a79de71bd969ef5bf944fd9f46430338ff7e
  Author: Peng Liu <winner245 at hotmail.com>
  Date:   2024-12-19 (Thu, 19 Dec 2024)

  Changed paths:
    A libcxx/test/std/containers/sequences/vector.bool/assign_iter_iter.pass.cpp
    A libcxx/test/std/containers/sequences/vector.bool/assign_size_value.pass.cpp
    M libcxx/test/std/containers/sequences/vector/vector.cons/assign_iter_iter.pass.cpp

  Log Message:
  -----------
  [libc++][test] Improve tests for assign in std::vector and vector<bool> (#119163)

This PR enhances the test coverage for std::vector::assign by adding new
tests for several important test cases that were previously missing, as
shown in the following table:

| test cases                        | forward_iterator | input_iterator |
|-----------------------------------|------------------|----------------|
| new_size > capacity()             | Yes              | Yes            |
| size() < new_size <= capacity()   | No               | No             |
| new_size <= size()                | No               | No             |

Similarly, no tests have previously covered `assign(InputIterator, InputIterator)`
and `assign(size_type, const value_type&)` for `vector<bool>`.

With this patch applied, all missing tests are covered.



To unsubscribe from these emails, change your notification settings at https://github.com/llvm/llvm-project/settings/notifications


More information about the All-commits mailing list