[all-commits] [llvm/llvm-project] 3784bd: [libcxx] [test] Fix string type handling in a few ...

Martin Storsjö via All-commits all-commits at lists.llvm.org
Fri Oct 16 11:05:04 PDT 2020


  Branch: refs/heads/master
  Home:   https://github.com/llvm/llvm-project
  Commit: 3784bdf2176f38cc30134fab776efb43506c0c54
      https://github.com/llvm/llvm-project/commit/3784bdf2176f38cc30134fab776efb43506c0c54
  Author: Martin Storsjö <martin at martin.st>
  Date:   2020-10-16 (Fri, 16 Oct 2020)

  Changed paths:
    M libcxx/test/std/input.output/filesystems/class.path/path.member/path.assign/copy.pass.cpp
    M libcxx/test/std/input.output/filesystems/class.path/path.member/path.assign/move.pass.cpp
    M libcxx/test/std/input.output/filesystems/class.path/path.member/path.construct/copy.pass.cpp
    M libcxx/test/std/input.output/filesystems/class.path/path.member/path.construct/move.pass.cpp
    M libcxx/test/std/input.output/filesystems/class.path/path.member/path.construct/source.pass.cpp
    M libcxx/test/std/input.output/filesystems/class.path/path.member/path.native.obs/c_str.pass.cpp
    M libcxx/test/std/input.output/filesystems/class.path/path.member/path.native.obs/native.pass.cpp
    M libcxx/test/std/input.output/filesystems/class.path/path.member/path.native.obs/operator_string.pass.cpp
    M libcxx/test/std/input.output/filesystems/class.path/path.nonmember/path.io.pass.cpp
    M libcxx/test/std/input.output/filesystems/class.path/path.nonmember/swap.pass.cpp
    M libcxx/test/std/input.output/filesystems/class.path/synop.pass.cpp

  Log Message:
  -----------
  [libcxx] [test] Fix string type handling in a few fairly trivial class.path tests

Use string() for convenience for testing where possible, but keep using
native() for move tests where we want to check that no allocations are
made, constructing a reference fs::path::string_type instead.

Use the right value_type in a few places.

Make the synop test check for the right types and for the expected
preferred separator.

Differential Revision: https://reviews.llvm.org/D89537




More information about the All-commits mailing list