diff options
author | Jonathan Wakely <jwakely@redhat.com> | 2019-04-30 11:39:59 +0100 |
---|---|---|
committer | Jonathan Wakely <redi@gcc.gnu.org> | 2019-04-30 11:39:59 +0100 |
commit | 95767c6597e32256b3850c9bed4fd8ce1f0da089 (patch) | |
tree | 0bd4a9c2babb4798a8d651ac383663e51622d0f7 | |
parent | 0d67a51060e7d67b685ace6ba5395d9a98e2f5be (diff) | |
download | gcc-95767c6597e32256b3850c9bed4fd8ce1f0da089.zip gcc-95767c6597e32256b3850c9bed4fd8ce1f0da089.tar.gz gcc-95767c6597e32256b3850c9bed4fd8ce1f0da089.tar.bz2 |
Fix filesystem::path tests
The root_path.cc test had some debugging macros left in accidentally, so
didn't FAIL correctly if an assertion failed.
The string-char8_t.cc tests didn't compile on Windows.
* testsuite/27_io/filesystem/path/decompose/root_path.cc: Remove
macros accidentally left in.
* testsuite/27_io/filesystem/path/native/string-char8_t.cc: Remove
unnecessary -lstdc++fs option. Fix test for mingw.
* testsuite/experimental/filesystem/path/native/string-char8_t.cc:
Fix test for mingw.
From-SVN: r270685
4 files changed, 18 insertions, 12 deletions
diff --git a/libstdc++-v3/ChangeLog b/libstdc++-v3/ChangeLog index 6798f4d..9ea672f 100644 --- a/libstdc++-v3/ChangeLog +++ b/libstdc++-v3/ChangeLog @@ -1,3 +1,12 @@ +2019-04-30 Jonathan Wakely <jwakely@redhat.com> + + * testsuite/27_io/filesystem/path/decompose/root_path.cc: Remove + macros accidentally left in. + * testsuite/27_io/filesystem/path/native/string-char8_t.cc: Remove + unnecessary -lstdc++fs option. Fix test for mingw. + * testsuite/experimental/filesystem/path/native/string-char8_t.cc: + Fix test for mingw. + 2019-04-30 Jakub Jelinek <jakub@redhat.com> * config/abi/post/riscv64-linux-gnu/baseline_symbols.txt: Update. diff --git a/libstdc++-v3/testsuite/27_io/filesystem/path/decompose/root_path.cc b/libstdc++-v3/testsuite/27_io/filesystem/path/decompose/root_path.cc index 4f4c531..d573b48 100644 --- a/libstdc++-v3/testsuite/27_io/filesystem/path/decompose/root_path.cc +++ b/libstdc++-v3/testsuite/27_io/filesystem/path/decompose/root_path.cc @@ -35,10 +35,6 @@ test01() VERIFY( p2.root_path() == path("/") ); } -#undef VERIFY -#define VERIFY(X) do { if (!(X)) { __builtin_puts("FAIL: " #X); } } while(false) -#define DUMP(X, Y, Z) do { if (!(Y == Z)) { __builtin_printf("%s %s %s\n", X.c_str(), Y.c_str(), Z.c_str()); } } while(false) - void test02() { @@ -48,7 +44,6 @@ test02() path rootn = p.root_name(); path rootd = p.root_directory(); VERIFY( rootp == (rootn / rootd) ); - DUMP(p, rootp , (rootn / rootd) ); } } diff --git a/libstdc++-v3/testsuite/27_io/filesystem/path/native/string-char8_t.cc b/libstdc++-v3/testsuite/27_io/filesystem/path/native/string-char8_t.cc index f5bb1af..d333787 100644 --- a/libstdc++-v3/testsuite/27_io/filesystem/path/native/string-char8_t.cc +++ b/libstdc++-v3/testsuite/27_io/filesystem/path/native/string-char8_t.cc @@ -15,7 +15,7 @@ // with this library; see the file COPYING3. If not see // <http://www.gnu.org/licenses/>. -// { dg-options "-std=gnu++17 -lstdc++fs -fchar8_t" } +// { dg-options "-std=gnu++17 -fchar8_t" } // { dg-do run { target c++17 } } // { dg-require-filesystem-ts "" } @@ -27,14 +27,15 @@ void test01() { using namespace std::filesystem; - const std::string s = "abc"; + using string_type = std::basic_string<path::value_type>; + const string_type s{ 'a', 'b', 'c' }; path p(s); VERIFY( p.native() == s ); VERIFY( p.c_str() == s ); - VERIFY( static_cast<std::string>(p) == s ); + VERIFY( static_cast<string_type>(p) == s ); - std::string s2 = p; // implicit conversion + string_type s2 = p; // implicit conversion VERIFY( s2 == p.native() ); } diff --git a/libstdc++-v3/testsuite/experimental/filesystem/path/native/string-char8_t.cc b/libstdc++-v3/testsuite/experimental/filesystem/path/native/string-char8_t.cc index a0d8058..091663f4 100644 --- a/libstdc++-v3/testsuite/experimental/filesystem/path/native/string-char8_t.cc +++ b/libstdc++-v3/testsuite/experimental/filesystem/path/native/string-char8_t.cc @@ -27,14 +27,15 @@ void test01() { using namespace std::experimental::filesystem; - const std::string s = "abc"; + using string_type = std::basic_string<path::value_type>; + const string_type s{ 'a', 'b', 'c' }; path p(s); VERIFY( p.native() == s ); VERIFY( p.c_str() == s ); - VERIFY( static_cast<std::string>(p) == s ); + VERIFY( static_cast<string_type>(p) == s ); - std::string s2 = p; // implicit conversion + string_type s2 = p; // implicit conversion VERIFY( s2 == p.native() ); } |