[PR #1039] [MERGED] Convert file offsets from size_t to off_t #1772

New issue

Closed

opened 2026-03-04 02:02:05 +03:00 by kerem · 0 comments

kerem commented

2026-03-04 02:02:05 +03:00

Owner

📋 Pull Request Information

Original PR: https://github.com/s3fs-fuse/s3fs-fuse/pull/1039
Author: @gaul
Created: 6/14/2019
Status: ✅ Merged
Merged: 6/23/2019
Merged by: @ggtakec

Base: master ← Head: off_t

📝 Commits (1)

15b7450 Convert file offsets from size_t to off_t

📊 Changes

5 files changed (+172 additions, -172 deletions)

View changed files

📝 src/curl.cpp (+7 -7)
📝 src/curl.h (+2 -2)
📝 src/fdcache.cpp (+121 -121)
📝 src/fdcache.h (+29 -29)
📝 src/s3fs.cpp (+13 -13)

📄 Description

The latter is 64-bits on 32-bit platforms when specifying
-D_FILE_OFFSET_BITS=64. This allows early Raspberry Pis to use files
larger than 2 GB. It also cleans up some ugly casting. Fixes #620.
Fixes #656.

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/s3fs-fuse/s3fs-fuse/pull/1039 **Author:** [@gaul](https://github.com/gaul) **Created:** 6/14/2019 **Status:** ✅ Merged **Merged:** 6/23/2019 **Merged by:** [@ggtakec](https://github.com/ggtakec) **Base:** `master` ← **Head:** `off_t` --- ### 📝 Commits (1) - [`15b7450`](https://github.com/s3fs-fuse/s3fs-fuse/commit/15b7450713ee2f776d46b383b97800ee43c28caf) Convert file offsets from size_t to off_t ### 📊 Changes **5 files changed** (+172 additions, -172 deletions) <details> <summary>View changed files</summary> 📝 `src/curl.cpp` (+7 -7) 📝 `src/curl.h` (+2 -2) 📝 `src/fdcache.cpp` (+121 -121) 📝 `src/fdcache.h` (+29 -29) 📝 `src/s3fs.cpp` (+13 -13) </details> ### 📄 Description The latter is 64-bits on 32-bit platforms when specifying `-D_FILE_OFFSET_BITS=64`. This allows early Raspberry Pis to use files larger than 2 GB. It also cleans up some ugly casting. Fixes #620. Fixes #656. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>