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

[Marshall Clow via Phabricator via cfe-commits]

mclow.lists added inline comments.


================
Comment at: test/std/experimental/filesystem/fs.op.funcs/fs.op.remove/remove.pass.cpp:67
         non_empty_dir,
-        file_in_bad_dir,
+        // file_in_bad_dir, // produces "St13exception_ptruncaught_exceptions not yet implemented"
     };
----------------
TyanNN wrote:
> This error is quite strange: it is not inherited from std::exception, one can only really find our the error with std::current_exception, and it happens when you use the path in fs::remove and fs::remove_all. Any idea about this? @mclow.lists
I'm not seeing this error on my system (nor are any of the bots reporting it AFAIK).

That looks like "`uncaught_exceptions` is not yet implemented", and that comes from the ABI library.

What are you using for an ABI library? Is it libc++abi?



https://reviews.llvm.org/D41830