Fixed as many clang-tidy include warnings as possible

eclipse-ecal · Nov 2, 2023 · 9802434 · 9802434
1 parent c148060
commit 9802434
Show file tree

Hide file tree

Showing 11 changed files with 73 additions and 46 deletions.
diff --git a/fineftp-server/include/fineftp/server.h b/fineftp-server/include/fineftp/server.h
@@ -1,8 +1,9 @@
 #pragma once
 
+#include <cstddef>
+#include <cstdint>
 #include <memory>
 #include <string>
-#include <cstdint>
 
 // IWYU pragma: begin_exports
 #include <fineftp/permissions.h>

diff --git a/fineftp-server/src/filesystem.cpp b/fineftp-server/src/filesystem.cpp
@@ -1,32 +1,35 @@
 #include "filesystem.h"
 
+#include <array>
+#include <cstdint>
+#include <iomanip>
 #include <list>
+#include <mutex> // IWYU pragma: keep
 #include <sstream>
-#include <mutex>
-#include <iomanip>
-#include <array>
 
 #include <chrono>
-#include <string>
+#include <ctime>
 #include <iostream>
 #include <map>
-#include <algorithm>
 #include <regex>
-#include <ctime>
+#include <string>
 
 #include <sys/stat.h>
 
 #ifdef WIN32
 
-#ifndef NOMINMAX
-#define NOMINMAX
-#endif
-#define WIN32_LEAN_AND_MEAN
-#include <windows.h>
-#include <win_str_convert.h>
+  #ifndef NOMINMAX
+    #define NOMINMAX
+  #endif
+
+  #define WIN32_LEAN_AND_MEAN
+
+  #include <windows.h>
+  #include <win_str_convert.h>
 
 #else // WIN32
 
+#include <cerrno>
 #include <cstring>
 #include <dirent.h>
 

diff --git a/fineftp-server/src/filesystem.h b/fineftp-server/src/filesystem.h
@@ -1,8 +1,6 @@
 #pragma once
 
-#include <chrono>
-#include <regex>
-#include <iostream>
+#include <cstdint>
 #include <map>
 #include <string>
 

diff --git a/fineftp-server/src/ftp_session.cpp b/fineftp-server/src/ftp_session.cpp
@@ -1,25 +1,31 @@
 #include "ftp_session.h"
 
-#include <iostream>
-#include <iomanip>
 #include <algorithm>
-#include <map>
-#include <functional>
+#include <cassert> // assert
+#include <cctype>  // std::iscntrl, toupper
+#include <cstddef>
+#include <cstdio>
 #include <fstream>
+#include <functional>
+#include <iomanip>
+#include <iostream>
+#include <iterator>
+#include <map>
+#include <memory>
 #include <sstream>
 #include <string>
+#include <sys/types.h>
+#include <unistd.h>
 #include <vector>
-#include <memory>
-#include <cstddef>
-#include <iterator>
-#include <cassert> // assert
-#include <cctype>  // std::iscntrl, toupper
 
 #include <file_man.h>
 
 #include "filesystem.h"
-#include <fineftp/permissions.h>
 #include "ftp_message.h"
+#include "user_database.h"
+#include <fineftp/permissions.h>
+
+#include <sys/stat.h>
 
 #ifdef WIN32
 #include "win_str_convert.h"

diff --git a/fineftp-server/src/ftp_session.h b/fineftp-server/src/ftp_session.h
@@ -1,20 +1,23 @@
 #pragma once
 
-#include <asio.hpp>
+#include <asio.hpp> // IWYU pragma: keep
 
 #include <deque>
-#include <string>
-#include <memory>
 #include <functional>
-
+#include <map>
+#include <memory>
+#include <string>
+#include <vector>
 
 #include "ftp_message.h"
 
 #include "filesystem.h"
 #include "user_database.h"
 #include "ftp_user.h"
 
-#include "win_str_convert.h"
+#ifdef WIN32
+  #include "win_str_convert.h"
+#endif // WIN32
 
 namespace fineftp
 {

diff --git a/fineftp-server/src/server_impl.h b/fineftp-server/src/server_impl.h
@@ -1,18 +1,18 @@
 #pragma once
 
-#include <vector>
-#include <string>
-#include <thread>
 #include <atomic>
-#include <cstdint>
 #include <cstddef>
+#include <cstdint>
+#include <memory>
+#include <string>
+#include <thread>
+#include <vector>
 
-#include <asio.hpp>
+#include <asio.hpp> // IWYU pragma: keep
 
-#include <ftp_session.h>
 #include <fineftp/permissions.h>
+#include <ftp_session.h>
 
-#include <ftp_user.h>
 #include <user_database.h>
 
 namespace fineftp

diff --git a/fineftp-server/src/unix/file_man.cpp b/fineftp-server/src/unix/file_man.cpp
@@ -2,12 +2,14 @@
 
 #include "file_man.h"
 
+#include <cstdint>
 #include <fcntl.h>
 #include <map>
+#include <memory>
 #include <mutex>
+#include <string>
 #include <sys/mman.h>
 #include <sys/stat.h>
-#include <sstream>
 #include <unistd.h>
 
 namespace fineftp

diff --git a/fineftp-server/src/unix/file_man.h b/fineftp-server/src/unix/file_man.h
@@ -3,8 +3,10 @@
 #ifndef FINEFTP_SERVER_SRC_UNIX_FILE_MAN_H_
 #define FINEFTP_SERVER_SRC_UNIX_FILE_MAN_H_
 
+#include <cstddef>
 #include <cstdint>
 #include <fstream>
+#include <ios>
 #include <memory>
 #include <string>
 #include <vector>

diff --git a/samples/fineftp_example/src/main.cpp b/samples/fineftp_example/src/main.cpp
@@ -1,8 +1,8 @@
+#include <chrono>
 #include <fineftp/server.h>
 
-#include <iostream>
-#include <thread>
 #include <string>
+#include <thread>
 
 int main() {
 

diff --git a/tests/fineftp_test/src/fineftp_stresstest.cpp b/tests/fineftp_test/src/fineftp_stresstest.cpp
@@ -1,12 +1,20 @@
+#include <chrono>
+#include <cstdlib>
+#include <functional>
 #include <gtest/gtest.h>
 
 #include <fineftp/server.h>
 
+#include <algorithm>
 #include <filesystem>
-#include <string>
 #include <fstream>
+#include <ios>
+#include <iostream>
+#include <iterator>
+#include <string>
+#include <system_error>
 #include <thread>
-#include <algorithm>
+#include <vector>
 
 #ifdef WIN32
 #include <win_str_convert.h>

diff --git a/tests/fineftp_test/src/permission_test.cpp b/tests/fineftp_test/src/permission_test.cpp
@@ -1,12 +1,16 @@
+#include <cstdint>
+#include <cstdlib>
 #include <gtest/gtest.h>
 
 #include <fineftp/server.h>
 
 #include <filesystem>
-#include <string>
 #include <fstream>
-#include <thread>
-#include <algorithm>
+#include <iterator>
+#include <string>
+#include <system_error>
+#include <utility>
+#include <vector>
 
 #ifdef WIN32
 #include <win_str_convert.h>