[PATCH] D22452: [libcxx] Fix last_write_time tests for filesystems that don't support negative and very large times.
Jonas Hahnfeld via cfe-commits
cfe-commits at lists.llvm.org
Mon Jul 18 00:18:12 PDT 2016
Hahnfeld added inline comments.
================
Comment at: test/std/experimental/filesystem/fs.op.funcs/fs.op.last_write_time/last_write_time.pass.cpp:89-95
@@ +88,9 @@
+ }
+ if (ec) {
+ assert(old_write_time == new_write_time);
+ return false;
+ } else {
+ assert(new_write_time <= -5);
+ return true;
+ }
+}
----------------
Obviously, I meant this part of the code, sorry:
> Hmm no, this doesn't work for me: `fs::last_write_time` just takes the negative timestamp and overflows somewhere internally...
https://reviews.llvm.org/D22452
More information about the cfe-commits
mailing list