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

Dan Albert via Phabricator reviews at reviews.llvm.org
Fri Jan 11 10:25:51 PST 2019


danalbert updated this revision to Diff 181314.
danalbert added a comment.

Added reference to spec in comment.


Repository:
  rCXX libc++

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D56578/new/

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,8 @@
   return posix_lstat(p, path_stat, ec);
 }
 
-bool posix_ftruncate(const FileDescriptor& fd, size_t to_size, error_code& ec) {
+// http://pubs.opengroup.org/onlinepubs/9699919799/functions/ftruncate.html
+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.181314.patch
Type: text/x-patch
Size: 568 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/libcxx-commits/attachments/20190111/025880da/attachment.bin>


More information about the libcxx-commits mailing list