llvm/libcxx
1
0
Fork 0
mirror of https://github.com/llvm-mirror/libcxx.git synced 2025-10-24 12:02:36 +08:00
Code Issues Packages Projects Releases Wiki Activity

Be more consistent about which bool value means an error occurred

Browse Source 
git-svn-id: https://llvm.org/svn/llvm-project/libcxx/trunk@338002 91177308-0d34-0410-b5e6-96231b3b80d8
This commit is contained in:
Eric Fiselier
2018-07-26 04:02:06 +00:00
parent d1759cf0e9
commit 8928a12b4e
1 changed files with 6 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files

12
src/experimental/filesystem/operations.cpp
View File

@@ -433,19 +433,19 @@ bool posix_ftruncate(const FileDescriptor& fd, size_t to_size,
error_code& ec) { error_code& ec) {
if (::ftruncate(fd.fd, to_size) == -1) { if (::ftruncate(fd.fd, to_size) == -1) {
ec = capture_errno(); ec = capture_errno();
return false; return true;
} }
ec.clear(); ec.clear();
return true; return false;
} }
bool posix_fchmod(const FileDescriptor& fd, const StatT& st, error_code& ec) { bool posix_fchmod(const FileDescriptor& fd, const StatT& st, error_code& ec) {
if (::fchmod(fd.fd, st.st_mode) == -1) { if (::fchmod(fd.fd, st.st_mode) == -1) {
ec = capture_errno(); ec = capture_errno();
return false; return true;
} }
ec.clear(); ec.clear();
return true; return false;
} }
bool stat_equivalent(const StatT& st1, const StatT& st2) { 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); return err.report(errc::bad_file_descriptor);
// Set the permissions and truncate the file we opened. // 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); 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); return err.report(m_ec);
} }