[PATCH] D56578: Fix size_t/off_t mixup in std::filesystem.

Dan Albert via Phabricator reviews at reviews.llvm.org
Thu Jan 10 21:40:35 PST 2019


danalbert created this revision.
danalbert added reviewers: EricWF, mclow.lists.
Herald added subscribers: ldionne, christof.

ftruncate takes an off_t, not a size_t.


Repository:
  rCXX libc++

https://reviews.llvm.org/D56578

Files:
  src/filesystem/operations.cpp


Index: src/filesystem/operations.cpp
===================================================================
--- src/filesystem/operations.cpp
+++ src/filesystem/operations.cpp
@@ -439,7 +439,7 @@
   return posix_lstat(p, path_stat, ec);
 }
 
-bool posix_ftruncate(const FileDescriptor& fd, size_t to_size, error_code& ec) {
+bool posix_ftruncate(const FileDescriptor& fd, off_t to_size, error_code& ec) {
   if (::ftruncate(fd.fd, to_size) == -1) {
     ec = capture_errno();
     return true;


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D56578.181209.patch
Type: text/x-patch
Size: 491 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/libcxx-commits/attachments/20190111/5dcb4b19/attachment.bin>


More information about the libcxx-commits mailing list