libstdc++: Fix -Wrange-loop-construct warnings in filesystem tests
authorJonathan Wakely <jwakely@redhat.com>
Fri, 27 Nov 2020 12:15:44 +0000 (12:15 +0000)
committerJonathan Wakely <jwakely@redhat.com>
Fri, 27 Nov 2020 13:34:22 +0000 (13:34 +0000)
Many tests do `for (const path& p : test_paths)` where test_paths is an
array of strings. To avoid -Wrange-loop-construct warnings the loop
variable should be an object, not a reference bound to a temporary.

libstdc++-v3/ChangeLog:

* testsuite/27_io/filesystem/operations/absolute.cc: Avoid
-Wrange-loop-construct warning.
* testsuite/27_io/filesystem/path/append/source.cc: Likewise.
* testsuite/27_io/filesystem/path/assign/copy.cc: Likewise.
* testsuite/27_io/filesystem/path/compare/path.cc: Likewise.
* testsuite/27_io/filesystem/path/construct/copy.cc: Likewise.
* testsuite/27_io/filesystem/path/decompose/extension.cc:
Likewise.
* testsuite/27_io/filesystem/path/decompose/filename.cc:
Likewise.
* testsuite/27_io/filesystem/path/decompose/parent_path.cc:
Likewise.
* testsuite/27_io/filesystem/path/decompose/relative_path.cc:
Likewise.
* testsuite/27_io/filesystem/path/decompose/root_directory.cc:
Likewise.
* testsuite/27_io/filesystem/path/decompose/root_path.cc:
Likewise.
* testsuite/27_io/filesystem/path/itr/traversal.cc: Likewise.
* testsuite/27_io/filesystem/path/modifiers/remove_filename.cc:
Likewise.
* testsuite/27_io/filesystem/path/modifiers/replace_extension.cc:
Likewise.
* testsuite/27_io/filesystem/path/modifiers/replace_filename.cc:
Likewise.
* testsuite/27_io/filesystem/path/nonmember/append.cc: Likewise.
* testsuite/27_io/filesystem/path/nonmember/cmp.cc: Likewise.
* testsuite/27_io/filesystem/path/nonmember/cmp_c++20.cc:
Likewise.
* testsuite/27_io/filesystem/path/nonmember/hash_value.cc:
Likewise.
* testsuite/27_io/filesystem/path/query/has_extension.cc:
Likewise.
* testsuite/27_io/filesystem/path/query/has_filename.cc:
Likewise.
* testsuite/27_io/filesystem/path/query/has_parent_path.cc:
Likewise.
* testsuite/27_io/filesystem/path/query/has_relative_path.cc:
Likewise.
* testsuite/27_io/filesystem/path/query/has_root_directory.cc:
Likewise.
* testsuite/27_io/filesystem/path/query/has_root_name.cc:
Likewise.
* testsuite/27_io/filesystem/path/query/has_root_path.cc:
Likewise.
* testsuite/27_io/filesystem/path/query/has_stem.cc: Likewise.
* testsuite/27_io/filesystem/path/query/is_relative.cc: Likewise.
* testsuite/experimental/filesystem/operations/absolute.cc:
Likewise.
* testsuite/experimental/filesystem/path/assign/copy.cc: Likewise.
* testsuite/experimental/filesystem/path/compare/path.cc:
Likewise.
* testsuite/experimental/filesystem/path/construct/copy.cc:
Likewise.
* testsuite/experimental/filesystem/path/decompose/extension.cc:
Likewise.
* testsuite/experimental/filesystem/path/decompose/filename.cc:
Likewise.
* testsuite/experimental/filesystem/path/decompose/parent_path.cc:
Likewise.
* testsuite/experimental/filesystem/path/decompose/relative_path.cc:
Likewise.
* testsuite/experimental/filesystem/path/decompose/root_directory.cc:
Likewise.
* testsuite/experimental/filesystem/path/decompose/root_path.cc:
Likewise.
* testsuite/experimental/filesystem/path/itr/traversal.cc:
Likewise.
* testsuite/experimental/filesystem/path/modifiers/remove_filename.cc:
Likewise.
* testsuite/experimental/filesystem/path/modifiers/replace_extension.cc:
Likewise.
* testsuite/experimental/filesystem/path/modifiers/replace_filename.cc:
Likewise.
* testsuite/experimental/filesystem/path/nonmember/hash_value.cc:
Likewise.
* testsuite/experimental/filesystem/path/query/has_extension.cc:
Likewise.
* testsuite/experimental/filesystem/path/query/has_filename.cc:
Likewise.
* testsuite/experimental/filesystem/path/query/has_parent_path.cc:
Likewise.
* testsuite/experimental/filesystem/path/query/has_relative_path.cc:
Likewise.
* testsuite/experimental/filesystem/path/query/has_root_directory.cc:
Likewise.
* testsuite/experimental/filesystem/path/query/has_root_name.cc:
Likewise.
* testsuite/experimental/filesystem/path/query/has_root_path.cc:
Likewise.
* testsuite/experimental/filesystem/path/query/has_stem.cc:
Likewise.
* testsuite/experimental/filesystem/path/query/is_relative.cc:
Likewise.

52 files changed:
libstdc++-v3/testsuite/27_io/filesystem/operations/absolute.cc
libstdc++-v3/testsuite/27_io/filesystem/path/append/source.cc
libstdc++-v3/testsuite/27_io/filesystem/path/assign/copy.cc
libstdc++-v3/testsuite/27_io/filesystem/path/compare/path.cc
libstdc++-v3/testsuite/27_io/filesystem/path/construct/copy.cc
libstdc++-v3/testsuite/27_io/filesystem/path/decompose/extension.cc
libstdc++-v3/testsuite/27_io/filesystem/path/decompose/filename.cc
libstdc++-v3/testsuite/27_io/filesystem/path/decompose/parent_path.cc
libstdc++-v3/testsuite/27_io/filesystem/path/decompose/relative_path.cc
libstdc++-v3/testsuite/27_io/filesystem/path/decompose/root_directory.cc
libstdc++-v3/testsuite/27_io/filesystem/path/decompose/root_path.cc
libstdc++-v3/testsuite/27_io/filesystem/path/itr/traversal.cc
libstdc++-v3/testsuite/27_io/filesystem/path/modifiers/remove_filename.cc
libstdc++-v3/testsuite/27_io/filesystem/path/modifiers/replace_extension.cc
libstdc++-v3/testsuite/27_io/filesystem/path/modifiers/replace_filename.cc
libstdc++-v3/testsuite/27_io/filesystem/path/nonmember/append.cc
libstdc++-v3/testsuite/27_io/filesystem/path/nonmember/cmp.cc
libstdc++-v3/testsuite/27_io/filesystem/path/nonmember/cmp_c++20.cc
libstdc++-v3/testsuite/27_io/filesystem/path/nonmember/hash_value.cc
libstdc++-v3/testsuite/27_io/filesystem/path/query/has_extension.cc
libstdc++-v3/testsuite/27_io/filesystem/path/query/has_filename.cc
libstdc++-v3/testsuite/27_io/filesystem/path/query/has_parent_path.cc
libstdc++-v3/testsuite/27_io/filesystem/path/query/has_relative_path.cc
libstdc++-v3/testsuite/27_io/filesystem/path/query/has_root_directory.cc
libstdc++-v3/testsuite/27_io/filesystem/path/query/has_root_name.cc
libstdc++-v3/testsuite/27_io/filesystem/path/query/has_root_path.cc
libstdc++-v3/testsuite/27_io/filesystem/path/query/has_stem.cc
libstdc++-v3/testsuite/27_io/filesystem/path/query/is_relative.cc
libstdc++-v3/testsuite/experimental/filesystem/operations/absolute.cc
libstdc++-v3/testsuite/experimental/filesystem/path/assign/copy.cc
libstdc++-v3/testsuite/experimental/filesystem/path/compare/path.cc
libstdc++-v3/testsuite/experimental/filesystem/path/construct/copy.cc
libstdc++-v3/testsuite/experimental/filesystem/path/decompose/extension.cc
libstdc++-v3/testsuite/experimental/filesystem/path/decompose/filename.cc
libstdc++-v3/testsuite/experimental/filesystem/path/decompose/parent_path.cc
libstdc++-v3/testsuite/experimental/filesystem/path/decompose/relative_path.cc
libstdc++-v3/testsuite/experimental/filesystem/path/decompose/root_directory.cc
libstdc++-v3/testsuite/experimental/filesystem/path/decompose/root_path.cc
libstdc++-v3/testsuite/experimental/filesystem/path/itr/traversal.cc
libstdc++-v3/testsuite/experimental/filesystem/path/modifiers/remove_filename.cc
libstdc++-v3/testsuite/experimental/filesystem/path/modifiers/replace_extension.cc
libstdc++-v3/testsuite/experimental/filesystem/path/modifiers/replace_filename.cc
libstdc++-v3/testsuite/experimental/filesystem/path/nonmember/hash_value.cc
libstdc++-v3/testsuite/experimental/filesystem/path/query/has_extension.cc
libstdc++-v3/testsuite/experimental/filesystem/path/query/has_filename.cc
libstdc++-v3/testsuite/experimental/filesystem/path/query/has_parent_path.cc
libstdc++-v3/testsuite/experimental/filesystem/path/query/has_relative_path.cc
libstdc++-v3/testsuite/experimental/filesystem/path/query/has_root_directory.cc
libstdc++-v3/testsuite/experimental/filesystem/path/query/has_root_name.cc
libstdc++-v3/testsuite/experimental/filesystem/path/query/has_root_path.cc
libstdc++-v3/testsuite/experimental/filesystem/path/query/has_stem.cc
libstdc++-v3/testsuite/experimental/filesystem/path/query/is_relative.cc

index 96ccd777af412ba5b105bf19429e1a2721e261c0..712d131dfaa8572f426e78793e1b29f147bd3691 100644 (file)
@@ -30,7 +30,7 @@ using std::filesystem::path;
 void
 test01()
 {
-  for (const path& p : __gnu_test::test_paths)
+  for (const path p : __gnu_test::test_paths)
   {
     std::error_code ec;
     path abs = absolute(p, ec);
index dc7331945fe9ff6565cba77d66d8e0e0e7e613c2..4468a24efe7363bfd5b629c2f8be22107a7d3c6b 100644 (file)
@@ -92,8 +92,8 @@ test02()
 void
 test03()
 {
-  for (const path& p : __gnu_test::test_paths)
-    for (const path& q : __gnu_test::test_paths)
+  for (const path p : __gnu_test::test_paths)
+    for (const path q : __gnu_test::test_paths)
     {
       test(p, q.c_str());
       if constexpr (!std::is_same_v<path::value_type, char>)
index 0f912a5f74d73ee905824a605fe594df4b106b78..c0fd2aa3d9dcf0854eb00f325072d0029e7bc137 100644 (file)
@@ -28,7 +28,7 @@ using __gnu_test::compare_paths;
 void
 test01()
 {
-  for (const path& p : __gnu_test::test_paths)
+  for (const path p : __gnu_test::test_paths)
   {
     path copy;
     copy = p;
@@ -39,7 +39,7 @@ test01()
 void
 test02()
 {
-  for (const path& p : __gnu_test::test_paths)
+  for (const path p : __gnu_test::test_paths)
   {
     path copy = p;
     path move;
index 6f4166b641db823f64670ca07fd4775575521ba4..fd310fb827b41f1b9e63c04d7ceb244a156b1d0c 100644 (file)
@@ -30,7 +30,7 @@ void
 test01()
 {
   const path p0 = "/a/a/b/b";
-  for (const path& p : __gnu_test::test_paths)
+  for (const path p : __gnu_test::test_paths)
   {
     VERIFY( p.compare(p) == 0 );
     int cmp = p.compare(p0);
index 54b3510b9dee97ae336a1343dcc950fcfb41cfc8..a61c7c4f60955c0af7a2e900f17c6e2bc18966b0 100644 (file)
@@ -28,7 +28,7 @@ using std::filesystem::path;
 void
 test01()
 {
-  for (const path& p : __gnu_test::test_paths)
+  for (const path p : __gnu_test::test_paths)
   {
     path copy = p;
     __gnu_test::compare_paths(p, copy);
@@ -38,7 +38,7 @@ test01()
 void
 test02()
 {
-  for (const path& p : __gnu_test::test_paths)
+  for (const path p : __gnu_test::test_paths)
   {
     path copy = p;
     path move = std::move(copy);
index 032af98d9dd1c0437bf2086a59856341dcadbb88..94989706d1bbc55b87289748badcb48c8a787598 100644 (file)
@@ -51,7 +51,7 @@ test01()
 void
 test02()
 {
-  for (const path& p : __gnu_test::test_paths)
+  for (const path p : __gnu_test::test_paths)
   {
     auto stem = p.stem();
     auto ext = p.extension();
index 52dc3899cc7e35ab0613585a50229c3b5757aa05..2ab60144f73f34de2d9d3c103b8f7e4c2b7595e3 100644 (file)
@@ -46,7 +46,7 @@ test01()
 void
 test02()
 {
-  for (const path& p : __gnu_test::test_paths)
+  for (const path p : __gnu_test::test_paths)
   {
     path f = p.filename();
     if (p.empty())
index cf3724025f75b37583e7b0c797ded4935d6fd072..f198a4c8197dac328f239cf064a88fbe6606fb2e 100644 (file)
@@ -46,7 +46,7 @@ test01()
 void
 test02()
 {
-  for (const path& p : __gnu_test::test_paths)
+  for (const path p : __gnu_test::test_paths)
   {
     if (p.begin() == p.end())
       continue;
index 3d14829e0d760331a2e916e6ab44630352aed783..d514951649ead3a287c17460bd3c619500cbe0ec 100644 (file)
@@ -42,7 +42,7 @@ test01()
 void
 test02()
 {
-  for (const path& p : __gnu_test::test_paths)
+  for (const path p : __gnu_test::test_paths)
   {
     bool after_root = false;
     const path prel = p.relative_path();
index b89dd47bef94445fbf16fc5da6f70f81d4dc9698..6e24cc874c25d0995a66db140226050fbc1a6136 100644 (file)
@@ -46,7 +46,7 @@ test01()
 void
 test02()
 {
-  for (const path& p : __gnu_test::test_paths)
+  for (const path p : __gnu_test::test_paths)
   {
     path rootdir = p.root_directory();
     VERIFY( !rootdir.has_relative_path() );
index 10ed3144fa2613e8de099957dba4fcbefdea0e1e..aa0516f8aa3c510725f0d105c4a58386e8c775a9 100644 (file)
@@ -38,7 +38,7 @@ test01()
 void
 test02()
 {
-  for (const path& p : __gnu_test::test_paths)
+  for (const path p : __gnu_test::test_paths)
   {
     path rootp = p.root_path();
     path rootn = p.root_name();
index 43cb43b373daff8a3474684494676ce5b448fdff..b65f99758d3637848ed0c0c21741f43fadb9ed92 100644 (file)
@@ -108,7 +108,7 @@ test02()
   using reverse_iterator = std::reverse_iterator<path::iterator>;
   std::vector<path> fwd, rev;
 
-  for (const path& p : __gnu_test::test_paths)
+  for (const path p : __gnu_test::test_paths)
   {
     const auto begin = p.begin(), end = p.end();
     fwd.assign(begin, end);
index f47227a47e025c9ff53419441cd3bb6ea0f087b4..7848abd3f20bad63f97ae5a76234fd6ed67e06e6 100644 (file)
@@ -40,7 +40,7 @@ test01()
 void
 test02()
 {
-  for (const path& p : __gnu_test::test_paths)
+  for (const path p : __gnu_test::test_paths)
   {
     path p2(p);
     p2.remove_filename();
index a7b61f0b9f2422532bae7d9d777fcdfdf87d1a72..6cb23008a43863e5d1a4b3111a5516db2378b1f2 100644 (file)
@@ -47,7 +47,7 @@ test01()
 void
 test02()
 {
-  for (const path& p : __gnu_test::test_paths)
+  for (const path p : __gnu_test::test_paths)
   {
     path p2 = p;
     compare_paths( p2.replace_extension(p2.extension()), p );
index 48572ac8917b64e128bdc55bb90b21fd25e8e7a5..c8e39e2553208e38d25cd2a4aa9b0942c82f3ff8 100644 (file)
@@ -38,7 +38,7 @@ test01()
 void
 test02()
 {
-  for (const path& p : __gnu_test::test_paths)
+  for (const path p : __gnu_test::test_paths)
   {
     path p2(p);
     p2.replace_filename(p.filename());
index 03da11f0360491a0885096b83cad5be0ee106b21..b05f5f2d42c33d56be8c7e60611f74471e4ced71 100644 (file)
@@ -69,8 +69,8 @@ test02()
 void
 test03()
 {
-  for (const path& p : __gnu_test::test_paths)
-    for (const path& q : __gnu_test::test_paths)
+  for (const path p : __gnu_test::test_paths)
+    for (const path q : __gnu_test::test_paths)
       test(p, q);
 }
 
index d9adfad0a3e66799f5b81785977eb0f7be30998a..ff3741c33824a35a250a1a53109fb0cfdd27e206 100644 (file)
@@ -47,7 +47,7 @@ void
 test02()
 {
   const path p0 = "/a/a/b/b";
-  for (const path& p : __gnu_test::test_paths)
+  for (const path p : __gnu_test::test_paths)
   {
     VERIFY( p.compare(p) == 0 );
     int cmp = p.compare(p0);
index 34e8bf841c23ea6254a267a037fb0ea776decf5c..786634decb367a264d4f99dc4814a20076ca862d 100644 (file)
@@ -53,7 +53,7 @@ void
 test02()
 {
   const path p0 = "/a/a/b/b";
-  for (const path& p : __gnu_test::test_paths)
+  for (const path p : __gnu_test::test_paths)
   {
     VERIFY( std::is_eq(p <=> p) );
     VERIFY( (p <=> p0) == (p.compare(p0) <=> 0) );
index d665bcb74a24f5c410d6d6e0e5b72e10b32a434e..0164fa1bd0cbd85ecbeaf5a9d00a3d187123730a 100644 (file)
@@ -36,7 +36,7 @@ test01()
 void
 test02()
 {
-  for (const path& p : __gnu_test::test_paths)
+  for (const path p : __gnu_test::test_paths)
   {
     path pp = p.native();
     VERIFY( hash_value(p) == hash_value(pp) );
index 707e7acb60534e9a4936fbd1c859207cc272abf5..201406fc5af54d9e73952bc54fb6be22c3a26e05 100644 (file)
@@ -29,7 +29,7 @@ using std::filesystem::path;
 void
 test01()
 {
-  for (const path& p : __gnu_test::test_paths)
+  for (const path p : __gnu_test::test_paths)
   {
     VERIFY( p.has_extension() == !p.extension().empty() );
   }
index b1848d7761d620891c1964ff6807dc5b7cec14db..f6287cbcf4de42899724bc3ee292e2f1d124047e 100644 (file)
@@ -29,7 +29,7 @@ using std::filesystem::path;
 void
 test01()
 {
-  for (const path& p : __gnu_test::test_paths)
+  for (const path p : __gnu_test::test_paths)
   {
     VERIFY( p.has_filename() == !p.filename().empty() );
   }
index 7f34be268ca162b5d5983e68726cc67e59f9bfc5..3013dbf303a13f3399dfeec37248e9dc0c7c9603 100644 (file)
@@ -29,7 +29,7 @@ using std::filesystem::path;
 void
 test01()
 {
-  for (const path& p : __gnu_test::test_paths)
+  for (const path p : __gnu_test::test_paths)
   {
     VERIFY( p.has_parent_path() == !p.parent_path().empty() );
   }
index ef606b3a76429b990ccf414465c8fb5a8966daf9..3d44874a83f8e5f649aec984422a9d6ffc4b949d 100644 (file)
@@ -29,7 +29,7 @@ using std::filesystem::path;
 void
 test01()
 {
-  for (const path& p : __gnu_test::test_paths)
+  for (const path p : __gnu_test::test_paths)
   {
     VERIFY( p.has_relative_path() == !p.relative_path().empty() );
   }
index 5a35dedf165819939b60810137042fb6eec1f1ea..741b409b1eed2ec33a65a5b471e21d8d5e686895 100644 (file)
@@ -29,7 +29,7 @@ using std::filesystem::path;
 void
 test01()
 {
-  for (const path& p : __gnu_test::test_paths)
+  for (const path p : __gnu_test::test_paths)
   {
     VERIFY( p.has_root_directory() == !p.root_directory().empty() );
   }
index 0459873f76cb278a90385a9b2230c6e6fd7e4636..079f9c9314fc3068de186e41e7efdd3b4e8f4009 100644 (file)
@@ -29,7 +29,7 @@ using std::filesystem::path;
 void
 test01()
 {
-  for (const path& p : __gnu_test::test_paths)
+  for (const path p : __gnu_test::test_paths)
   {
     VERIFY( p.has_root_name() == !p.root_name().empty() );
   }
index 99c6951ebb6c934e799a240a15d8900992600800..19f1c8f0d2913583f3fafacf7617a4aa480e302f 100644 (file)
@@ -29,7 +29,7 @@ using std::filesystem::path;
 void
 test01()
 {
-  for (const path& p : __gnu_test::test_paths)
+  for (const path p : __gnu_test::test_paths)
   {
     VERIFY( p.has_root_path() == !p.root_path().empty() );
   }
index c5318990f154ce464133ee24f6950deba1cba9fe..7893baf23bc657892526914ae97f386629d96d0e 100644 (file)
@@ -29,7 +29,7 @@ using std::filesystem::path;
 void
 test01()
 {
-  for (const path& p : __gnu_test::test_paths)
+  for (const path p : __gnu_test::test_paths)
   {
     VERIFY( p.has_stem() == !p.stem().empty() );
   }
index ba6fe3cbb0c64d979f7138556a08b053a385b451..0ee4d94497439265348461e5101e75d449999291 100644 (file)
@@ -29,7 +29,7 @@ using std::filesystem::path;
 void
 test01()
 {
-  for (const path& p : __gnu_test::test_paths)
+  for (const path p : __gnu_test::test_paths)
   {
     VERIFY( p.is_relative() == !p.is_absolute() );
   }
index 5c9627f63b6438962b0b4eb19397cd6fd1eda6df..8b9fb73665705d5ff1d8228d5d4f98f97c86051a 100644 (file)
@@ -30,7 +30,7 @@ using std::experimental::filesystem::path;
 void
 test01()
 {
-  for (const path& p : __gnu_test::test_paths)
+  for (const path p : __gnu_test::test_paths)
   {
 #if defined(__MINGW32__) || defined(__MINGW64__)
     if (p.empty())
index 2f79360032423a36718c05076827104ccf4bf0f1..09e2fd6e5e604b11f49a990809be4d080f54989d 100644 (file)
@@ -28,7 +28,7 @@ using __gnu_test::compare_paths;
 void
 test01()
 {
-  for (const path& p : __gnu_test::test_paths)
+  for (const path p : __gnu_test::test_paths)
   {
     path copy;
     copy = p;
@@ -39,7 +39,7 @@ test01()
 void
 test02()
 {
-  for (const path& p : __gnu_test::test_paths)
+  for (const path p : __gnu_test::test_paths)
   {
     path copy = p;
     path move;
index 5228d0a8ca84d1c7afba53bf969d17f931b3c906..66606f7dca07eb6c68575eee1de413718f473a59 100644 (file)
@@ -31,7 +31,7 @@ void
 test01()
 {
   const path p0 = "/a/a/b/b";
-  for (const path& p : __gnu_test::test_paths)
+  for (const path p : __gnu_test::test_paths)
   {
     VERIFY( p.compare(p) == 0 );
     int cmp = p.compare(p0);
index f075ba65e8a33641de1eeaee5fe84434ef58571d..50a00c55d5a9ce4b548ac46827f4bfac1ba326bd 100644 (file)
@@ -29,7 +29,7 @@ using std::experimental::filesystem::path;
 void
 test01()
 {
-  for (const path& p : __gnu_test::test_paths)
+  for (const path p : __gnu_test::test_paths)
   {
     path copy = p;
     __gnu_test::compare_paths(p, copy);
@@ -39,7 +39,7 @@ test01()
 void
 test02()
 {
-  for (const path& p : __gnu_test::test_paths)
+  for (const path p : __gnu_test::test_paths)
   {
     path copy = p;
     path move = std::move(copy);
index 2d0b0882960464ad8b89576117bbc20986462f60..0c5b5f4fe78c10355aafc239778368b3db2e034c 100644 (file)
@@ -44,7 +44,7 @@ test01()
 void
 test02()
 {
-  for (const path& p : __gnu_test::test_paths)
+  for (const path p : __gnu_test::test_paths)
   {
     auto stem = p.stem();
     auto ext = p.extension();
index 066d7dcaba714cf67a72f0384f26e5bdebddbb02..a6a7b8d2c79d5864748036f4f5cd0d94ca513dd6 100644 (file)
@@ -39,7 +39,7 @@ test01()
 void
 test02()
 {
-  for (const path& p : __gnu_test::test_paths)
+  for (const path p : __gnu_test::test_paths)
   {
     path f = p.filename();
     if (p.empty())
index 3f12fad0bcd6b5a318dd87a10c783b1aaacc836e..bd3961a70cafe9ff7ce610aa47d02e9906746729 100644 (file)
@@ -43,7 +43,7 @@ test01()
 void
 test02()
 {
-  for (const path& p : __gnu_test::test_paths)
+  for (const path p : __gnu_test::test_paths)
   {
     if (p.begin() == p.end())
       continue;
index 43b5091bb65fa940fb46b04bb4bb61472f9823dd..14221de33899b5849fa16f614ac3de0316a7111a 100644 (file)
@@ -41,7 +41,7 @@ test01()
 void
 test02()
 {
-  for (const path& p : __gnu_test::test_paths)
+  for (const path p : __gnu_test::test_paths)
   {
     bool after_root = false;
     const path prel = p.relative_path();
index bbb500916e2133d123620eb0710751e5e62cacf9..ffc4839574733528902a236051d2ae0d704a6128 100644 (file)
@@ -43,7 +43,7 @@ test01()
 void
 test02()
 {
-  for (const path& p : __gnu_test::test_paths)
+  for (const path p : __gnu_test::test_paths)
   {
     path rootdir = p.root_directory();
     // If root-directory is composed of 'slash name',
index dfd2cb0da247ed097a86a370e87e3871ce3d9d53..7ffd526f02ac4c5837b2c464d0e38ee4b882efa8 100644 (file)
@@ -39,7 +39,7 @@ test01()
 void
 test02()
 {
-  for (const path& p : __gnu_test::test_paths)
+  for (const path p : __gnu_test::test_paths)
   {
     path rootp = p.root_path();
     path rootn = p.root_name();
index 83ee78b10aea029a6009a628a0330f364455990d..c1481777ed68f24c0c42675f5aa3709d4b73e7b0 100644 (file)
@@ -69,7 +69,7 @@ test02()
   using reverse_iterator = std::reverse_iterator<path::iterator>;
   std::vector<path> fwd, rev;
 
-  for (const path& p : __gnu_test::test_paths)
+  for (const path p : __gnu_test::test_paths)
   {
     const auto begin = p.begin(), end = p.end();
     fwd.assign(begin, end);
index 0537bf45461b4a17a5f2f1b0fa217a67b573f27c..5186c3d3105ac2d81164c16e685009bd4a408a9a 100644 (file)
@@ -37,7 +37,7 @@ test01()
 void
 test02()
 {
-  for (const path& p : __gnu_test::test_paths)
+  for (const path p : __gnu_test::test_paths)
   {
     path p2(p);
     p2.remove_filename();
index 5053c72724f36d21f6080175eb9718304d04dc2a..b09845447312cfec078920e3de578dc136f9ffc7 100644 (file)
@@ -38,7 +38,7 @@ test01()
 void
 test02()
 {
-  for (const path& p : __gnu_test::test_paths)
+  for (const path p : __gnu_test::test_paths)
   {
     path p2 = p;
     VERIFY(p2.replace_extension(p2.extension()) == p);
index 1d063b6e23f49f2970b968ad77508635ffdd303c..858ecd98303575565e3139ec8ed160f72ab0ce19 100644 (file)
@@ -37,7 +37,7 @@ test01()
 void
 test02()
 {
-  for (const path& p : __gnu_test::test_paths)
+  for (const path p : __gnu_test::test_paths)
   {
     path p2(p);
     p2.replace_filename(p.filename());
index 68534912ff0059336bade05d38df58667577e623..931727fce9ad91aa86cf5daaf1efecb02625ca70 100644 (file)
@@ -37,7 +37,7 @@ test01()
 void
 test02()
 {
-  for (const path& p : __gnu_test::test_paths)
+  for (const path p : __gnu_test::test_paths)
   {
     path pp = p.native();
     VERIFY( hash_value(p) == hash_value(pp) );
index 3041e106f4d1a0c2a1ca086588adc0be97ad6cec..031bc828120e4705f813aac4313a146789f7d8eb 100644 (file)
@@ -30,7 +30,7 @@ using std::experimental::filesystem::path;
 void
 test01()
 {
-  for (const path& p : __gnu_test::test_paths)
+  for (const path p : __gnu_test::test_paths)
   {
     VERIFY( p.has_extension() == !p.extension().empty() );
   }
index 5fa4c6e4e58a7044669961aee61bdd73abbfe869..5ad1e3d910f831c7bfd9670b7ce05fbf172011e5 100644 (file)
@@ -30,7 +30,7 @@ using std::experimental::filesystem::path;
 void
 test01()
 {
-  for (const path& p : __gnu_test::test_paths)
+  for (const path p : __gnu_test::test_paths)
   {
     VERIFY( p.has_filename() == !p.filename().empty() );
   }
index 5a16cf21b029a939db77920c24e617fabf69622b..99d0b775f710860899fe48178c48260f3348e14f 100644 (file)
@@ -30,7 +30,7 @@ using std::experimental::filesystem::path;
 void
 test01()
 {
-  for (const path& p : __gnu_test::test_paths)
+  for (const path p : __gnu_test::test_paths)
   {
     VERIFY( p.has_parent_path() == !p.parent_path().empty() );
   }
index 8674bda74b2ccbca3fcd9f124c068c401e6efc5c..fa58a81509294bddb37c996e81fe3f48a6e42136 100644 (file)
@@ -30,7 +30,7 @@ using std::experimental::filesystem::path;
 void
 test01()
 {
-  for (const path& p : __gnu_test::test_paths)
+  for (const path p : __gnu_test::test_paths)
   {
     VERIFY( p.has_relative_path() == !p.relative_path().empty() );
   }
index 262d7a4a2af1dda80864d3fa9c7e2b4daf7e5d17..930c9cb31dcefa878be0c3d8be8daaf67efe6518 100644 (file)
@@ -30,7 +30,7 @@ using std::experimental::filesystem::path;
 void
 test01()
 {
-  for (const path& p : __gnu_test::test_paths)
+  for (const path p : __gnu_test::test_paths)
   {
     VERIFY( p.has_root_directory() == !p.root_directory().empty() );
   }
index 05cc30679f04dda860eeea61c4be97aa3fc699f0..951a746fc3be3ab6eb42bbf9818711c23d3d7a3c 100644 (file)
@@ -30,7 +30,7 @@ using std::experimental::filesystem::path;
 void
 test01()
 {
-  for (const path& p : __gnu_test::test_paths)
+  for (const path p : __gnu_test::test_paths)
   {
     VERIFY( p.has_root_name() == !p.root_name().empty() );
   }
index bd771932f8a6617fe80c230aa4c25ec5ac6e3c37..501fc8285822d145aa3f09190d82750c1da48450 100644 (file)
@@ -30,7 +30,7 @@ using std::experimental::filesystem::path;
 void
 test01()
 {
-  for (const path& p : __gnu_test::test_paths)
+  for (const path p : __gnu_test::test_paths)
   {
     VERIFY( p.has_root_path() == !p.root_path().empty() );
   }
index acb70d955f1dad2a9ef39f44b57cf6412e55a41b..103aff0e28f91347b99480165f23c311ad7d9a58 100644 (file)
@@ -30,7 +30,7 @@ using std::experimental::filesystem::path;
 void
 test01()
 {
-  for (const path& p : __gnu_test::test_paths)
+  for (const path p : __gnu_test::test_paths)
   {
     VERIFY( p.has_stem() == !p.stem().empty() );
   }
index 6071400cb64b6dabb09a8b9ba42dacbcca9ad46c..86a60c6b8c8c2a0eeedb2499b618334916470102 100644 (file)
@@ -30,7 +30,7 @@ using std::experimental::filesystem::path;
 void
 test01()
 {
-  for (const path& p : __gnu_test::test_paths)
+  for (const path p : __gnu_test::test_paths)
   {
     VERIFY( p.is_relative() == !p.is_absolute() );
   }