[PR #1949] [MERGED] Fixed errors reported by cppcheck 2.8 #2267

New issue

Closed

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

kerem commented

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

Owner

📋 Pull Request Information

Original PR: https://github.com/s3fs-fuse/s3fs-fuse/pull/1949
Author: @ggtakec
Created: 5/25/2022
Status: ✅ Merged
Merged: 5/25/2022
Merged by: @ggtakec

Base: master ← Head: fix_cppcheck_28

📝 Commits (1)

96da31f Fixed errors reported by cppcheck 2.8

📊 Changes

7 files changed (+57 additions, -51 deletions)

View changed files

📝 src/bodydata.cpp (+1 -1)
📝 src/curl.cpp (+23 -20)
📝 src/curl_multi.cpp (+3 -3)
📝 src/s3fs.cpp (+4 -4)
📝 src/s3fs_util.cpp (+2 -2)
📝 src/s3fs_xml.cpp (+21 -21)
📝 src/sighandlers.cpp (+3 -0)

📄 Description

Relevant Issue (if applicable)

n/a

Details

It seems that the version of cppcheck has been upgraded from 2.7 to 2.8 on some macos runners.
Along with that, some errors were reported, so we took corrective action.
(Still, it doesn't seem to be all macos runners)

_{🔄 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/1949 **Author:** [@ggtakec](https://github.com/ggtakec) **Created:** 5/25/2022 **Status:** ✅ Merged **Merged:** 5/25/2022 **Merged by:** [@ggtakec](https://github.com/ggtakec) **Base:** `master` ← **Head:** `fix_cppcheck_28` --- ### 📝 Commits (1) - [`96da31f`](https://github.com/s3fs-fuse/s3fs-fuse/commit/96da31f0228a8baf8161e8e349f0093db1ab617a) Fixed errors reported by cppcheck 2.8 ### 📊 Changes **7 files changed** (+57 additions, -51 deletions) <details> <summary>View changed files</summary> 📝 `src/bodydata.cpp` (+1 -1) 📝 `src/curl.cpp` (+23 -20) 📝 `src/curl_multi.cpp` (+3 -3) 📝 `src/s3fs.cpp` (+4 -4) 📝 `src/s3fs_util.cpp` (+2 -2) 📝 `src/s3fs_xml.cpp` (+21 -21) 📝 `src/sighandlers.cpp` (+3 -0) </details> ### 📄 Description ### Relevant Issue (if applicable) n/a ### Details It seems that the version of cppcheck has been upgraded from 2.7 to 2.8 on some macos runners. Along with that, some errors were reported, so we took corrective action. _(Still, it doesn't seem to be all macos runners)_ --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>