Be more consistent about which bool value means an error occurred
authorEric Fiselier <eric@efcs.ca>
Thu, 26 Jul 2018 04:02:06 +0000 (04:02 +0000)
committerEric Fiselier <eric@efcs.ca>
Thu, 26 Jul 2018 04:02:06 +0000 (04:02 +0000)
llvm-svn: 338002

libcxx/src/experimental/filesystem/operations.cpp

index 028d5bf..775c178 100644 (file)
@@ -433,19 +433,19 @@ bool posix_ftruncate(const FileDescriptor& fd, size_t to_size,
                      error_code& ec) {
   if (::ftruncate(fd.fd, to_size) == -1) {
     ec = capture_errno();
-    return false;
+    return true;
   }
   ec.clear();
-  return true;
+  return false;
 }
 
 bool posix_fchmod(const FileDescriptor& fd, const StatT& st, error_code& ec) {
   if (::fchmod(fd.fd, st.st_mode) == -1) {
     ec = capture_errno();
-    return false;
+    return true;
   }
   ec.clear();
-  return true;
+  return false;
 }
 
 bool stat_equivalent(const StatT& st1, const StatT& st2) {
@@ -796,9 +796,9 @@ bool __copy_file(const path& from, const path& to, copy_options options,
       return err.report(errc::bad_file_descriptor);
 
     // Set the permissions and truncate the file we opened.
-    if (!detail::posix_fchmod(to_fd, from_stat, m_ec))
+    if (detail::posix_fchmod(to_fd, from_stat, m_ec))
       return err.report(m_ec);
-    if (!detail::posix_ftruncate(to_fd, 0, m_ec))
+    if (detail::posix_ftruncate(to_fd, 0, m_ec))
       return err.report(m_ec);
   }