[PR #2495] [MERGED] Wrap `CURL*` in a `std::unique_ptr` #2577

New issue

Closed

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

kerem commented

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

Owner

📋 Pull Request Information

Original PR: https://github.com/s3fs-fuse/s3fs-fuse/pull/2495
Author: @gaul
Created: 7/9/2024
Status: ✅ Merged
Merged: 7/15/2024
Merged by: @ggtakec

Base: master ← Head: unique_ptr/hCurl

📝 Commits (2)

a9177f9 Wrap CURL* in a std::unique_ptr
63c263f Merge branch 'master' into unique_ptr/hCurl

📊 Changes

4 files changed (+37 additions, -36 deletions)

View changed files

📝 src/curl.cpp (+20 -11)
📝 src/curl.h (+3 -3)
📝 src/curl_handlerpool.cpp (+9 -18)
📝 src/curl_handlerpool.h (+5 -4)

📄 Description

This is safer and clarifies the ownership of pointers.

_{🔄 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/2495 **Author:** [@gaul](https://github.com/gaul) **Created:** 7/9/2024 **Status:** ✅ Merged **Merged:** 7/15/2024 **Merged by:** [@ggtakec](https://github.com/ggtakec) **Base:** `master` ← **Head:** `unique_ptr/hCurl` --- ### 📝 Commits (2) - [`a9177f9`](https://github.com/s3fs-fuse/s3fs-fuse/commit/a9177f999e3656c938e0c8c38b31248b2ccbe092) Wrap CURL* in a std::unique_ptr - [`63c263f`](https://github.com/s3fs-fuse/s3fs-fuse/commit/63c263f76495f172d15c8071c9e30caf7676c51b) Merge branch 'master' into unique_ptr/hCurl ### 📊 Changes **4 files changed** (+37 additions, -36 deletions) <details> <summary>View changed files</summary> 📝 `src/curl.cpp` (+20 -11) 📝 `src/curl.h` (+3 -3) 📝 `src/curl_handlerpool.cpp` (+9 -18) 📝 `src/curl_handlerpool.h` (+5 -4) </details> ### 📄 Description This is safer and clarifies the ownership of pointers. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>