[PR #2553] [MERGED] Simplify some method parameters #2618

New issue

Closed

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

kerem commented

2026-03-04 02:06:27 +03:00

Owner

📋 Pull Request Information

Original PR: https://github.com/s3fs-fuse/s3fs-fuse/pull/2553
Author: @gaul
Created: 10/16/2024
Status: ✅ Merged
Merged: 10/17/2024
Merged by: @ggtakec

Base: master ← Head: params

📝 Commits (1)

f894fc5 Simplify some method parameters

📊 Changes

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

View changed files

📝 src/curl.cpp (+2 -2)
📝 src/curl.h (+1 -1)
📝 src/fdcache_fdinfo.cpp (+1 -1)
📝 src/fdcache_fdinfo.h (+1 -1)

📄 Description

Add const where possible and avoid unnecessary reference parameters.

_{🔄 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/2553 **Author:** [@gaul](https://github.com/gaul) **Created:** 10/16/2024 **Status:** ✅ Merged **Merged:** 10/17/2024 **Merged by:** [@ggtakec](https://github.com/ggtakec) **Base:** `master` ← **Head:** `params` --- ### 📝 Commits (1) - [`f894fc5`](https://github.com/s3fs-fuse/s3fs-fuse/commit/f894fc52a2d062c8369de28361252c9c52121a1a) Simplify some method parameters ### 📊 Changes **4 files changed** (+5 additions, -5 deletions) <details> <summary>View changed files</summary> 📝 `src/curl.cpp` (+2 -2) 📝 `src/curl.h` (+1 -1) 📝 `src/fdcache_fdinfo.cpp` (+1 -1) 📝 `src/fdcache_fdinfo.h` (+1 -1) </details> ### 📄 Description Add `const` where possible and avoid unnecessary reference parameters. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>