[PATCH] D41830: [libc++] Fix PR#35780 - make std::experimental::filesystem::remove and remove_all return false or 0 if the file doesn't exist

Eric Fiselier via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Jan 11 14:09:35 PST 2018


EricWF added inline comments.


================
Comment at: libcxx/trunk/test/std/experimental/filesystem/fs.op.funcs/fs.op.remove_all/remove_all.pass.cpp:86
+
+        TEST_CHECK(fs::remove_all(p) == 0);
+        TEST_CHECK(!ec);
----------------
This test is incorrect. `ec` isn't passed to the function.


Repository:
  rL LLVM

https://reviews.llvm.org/D41830





More information about the cfe-commits mailing list