mirror of
https://github.com/s3fs-fuse/s3fs-fuse.git
synced 2026-04-25 13:26:00 +03:00
[PR #2400] [MERGED] Fixed errors reported by cppcheck 2.13.0 #2520
Labels
No labels
bug
bug
dataloss
duplicate
enhancement
feature request
help wanted
invalid
need info
performance
pull-request
question
question
testing
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
starred/s3fs-fuse#2520
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
📋 Pull Request Information
Original PR: https://github.com/s3fs-fuse/s3fs-fuse/pull/2400
Author: @ggtakec
Created: 1/24/2024
Status: ✅ Merged
Merged: 1/24/2024
Merged by: @ggtakec
Base:
master← Head:fix_cppcheck213📝 Commits (1)
03e2160Fixed errors reported by cppcheck 2.13.0📊 Changes
8 files changed (+14 additions, -14 deletions)
View changed files
📝
src/fdcache.cpp(+2 -2)📝
src/fdcache_fdinfo.cpp(+1 -1)📝
src/fdcache_fdinfo.h(+1 -1)📝
src/s3fs.cpp(+6 -6)📝
src/s3fs_util.cpp(+1 -1)📝
src/string_util.cpp(+1 -1)📝
src/string_util.h(+1 -1)📝
src/test_curl_util.cpp(+1 -1)📄 Description
Relevant Issue (if applicable)
n/a
Details
Fixed an error reported in the new version cppcheck 2.13.0 (recommended use of const declaration).
NOTE
This fix will be merged immediately as it affects future fixes.
@gaul
If you have any problems with the fix, please let me know.
🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.