std-compat: remove #include:s which were added for pre C++17

these `#include`s were introduced by 0bbcbba and 79e846e, for backward compatibility with pre-C++17 era. since we've dropped C++17 support, and now only support C++20 and up. let's remove these cruft. Signed-off-by: Kefu Chai <[email protected]>
scylladb · May 21, 2024 · 9984358 · 9984358
1 parent 67a14f9
commit 9984358
Show file tree

Hide file tree

Showing 5 changed files with 4 additions and 8 deletions.
diff --git a/include/seastar/core/resource.hh b/include/seastar/core/resource.hh
@@ -27,6 +27,7 @@
 #ifndef SEASTAR_MODULE
 #include <cassert>
 #include <cstdlib>
+#include <optional>
 #include <string>
 #include <vector>
 #include <set>

diff --git a/include/seastar/util/exceptions.hh b/include/seastar/util/exceptions.hh
@@ -22,7 +22,7 @@
 
 #pragma once
 
-#include <seastar/util/std-compat.hh>
+#include <filesystem>
 
 namespace seastar {
 

diff --git a/include/seastar/util/read_first_line.hh b/include/seastar/util/read_first_line.hh
@@ -1,4 +1,4 @@
-#include <seastar/util/std-compat.hh>
+#include <filesystem>
 #include <seastar/core/sstring.hh>
 #ifndef SEASTAR_MODULE
 #include <boost/lexical_cast.hpp>

diff --git a/include/seastar/util/std-compat.hh b/include/seastar/util/std-compat.hh
@@ -25,12 +25,6 @@
 
 #ifndef SEASTAR_MODULE
 
-#include <optional>
-#include <string_view>
-#include <variant>
-
-#include <filesystem>
-
 #if __has_include(<memory_resource>)
 #include <memory_resource>
 #else

diff --git a/src/net/config.cc b/src/net/config.cc
@@ -29,6 +29,7 @@ module;
 #include <yaml-cpp/yaml.h>
 #include <algorithm>
 #include <istream>
+#include <optional>
 #include <unordered_map>
 #include <string>