[PR #1378] [MERGED] Source file division and set 4 spaces and cleanup #1947

New issue

Closed

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

kerem commented

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

Owner

📋 Pull Request Information

Original PR: https://github.com/s3fs-fuse/s3fs-fuse/pull/1378
Author: @ggtakec
Created: 8/22/2020
Status: ✅ Merged
Merged: 8/26/2020
Merged by: @gaul

Base: master ← Head: split_files

📝 Commits (1)

9b56c0f Source file division and set 4 spaces and cleanup

📊 Changes

72 files changed (+18457 additions, -16868 deletions)

View changed files

📝 Makefile.am (+9 -0)
📝 autogen.sh (+8 -0)
📝 configure.ac (+8 -0)
📝 src/Makefile.am (+38 -13)
📝 src/addhead.cpp (+180 -187)
📝 src/addhead.h (+29 -24)
➕ src/autolock.cpp (+82 -0)
➕ src/autolock.h (+60 -0)
➕ src/bodydata.cpp (+124 -0)
➕ src/bodydata.h (+72 -0)
📝 src/cache.cpp (+554 -564)
📝 src/cache.h (+129 -114)
📝 src/common.h (+13 -167)
📝 src/common_auth.cpp (+42 -40)
📝 src/curl.cpp (+3127 -3989)
📝 src/curl.h (+327 -495)
➕ src/curl_handlerpool.cpp (+129 -0)
➕ src/curl_handlerpool.h (+64 -0)
➕ src/curl_multi.cpp (+344 -0)
➕ src/curl_multi.h (+79 -0)

...and 52 more files

📄 Description

Relevant Issue (if applicable)

#1371

Details

Important

With this change, there is no change in logic or processing.
The purpose of this change is to clean up the source code.

Fix

The bloated source code was split(many files had beed splited).

Unified the source code indentation to 4-spaces.

Added header and footer to some files.

Other fixes

Global variable name changed("host"-> "s3host")
Some static function had been changed to global one
Modified types.h so that it can be built on other than C++11.

_{🔄 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/1378 **Author:** [@ggtakec](https://github.com/ggtakec) **Created:** 8/22/2020 **Status:** ✅ Merged **Merged:** 8/26/2020 **Merged by:** [@gaul](https://github.com/gaul) **Base:** `master` ← **Head:** `split_files` --- ### 📝 Commits (1) - [`9b56c0f`](https://github.com/s3fs-fuse/s3fs-fuse/commit/9b56c0f5e956f88b9a5fb2bf8ee8ade261eecbb0) Source file division and set 4 spaces and cleanup ### 📊 Changes **72 files changed** (+18457 additions, -16868 deletions) <details> <summary>View changed files</summary> 📝 `Makefile.am` (+9 -0) 📝 `autogen.sh` (+8 -0) 📝 `configure.ac` (+8 -0) 📝 `src/Makefile.am` (+38 -13) 📝 `src/addhead.cpp` (+180 -187) 📝 `src/addhead.h` (+29 -24) ➕ `src/autolock.cpp` (+82 -0) ➕ `src/autolock.h` (+60 -0) ➕ `src/bodydata.cpp` (+124 -0) ➕ `src/bodydata.h` (+72 -0) 📝 `src/cache.cpp` (+554 -564) 📝 `src/cache.h` (+129 -114) 📝 `src/common.h` (+13 -167) 📝 `src/common_auth.cpp` (+42 -40) 📝 `src/curl.cpp` (+3127 -3989) 📝 `src/curl.h` (+327 -495) ➕ `src/curl_handlerpool.cpp` (+129 -0) ➕ `src/curl_handlerpool.h` (+64 -0) ➕ `src/curl_multi.cpp` (+344 -0) ➕ `src/curl_multi.h` (+79 -0) _...and 52 more files_ </details> ### 📄 Description ### Relevant Issue (if applicable) #1371 ### Details #### Important With this change, there is no change in logic or processing. The purpose of this change is to clean up the source code. #### Fix ##### The bloated source code was split(many files had beed splited). ##### Unified the source code indentation to 4-spaces. ##### Added header and footer to some files. ##### Other fixes - Global variable name changed("host"-> "s3host") - Some static function had been changed to global one - Modified types.h so that it can be built on other than C++11. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>