Include cleanup

taocpp · Nov 15, 2024 · 6dfc53b · 6dfc53b
1 parent 4bd56be
commit 6dfc53b
Show file tree

Hide file tree

Showing 27 changed files with 37 additions and 19 deletions.
diff --git a/include/tao/pq/connection.hpp b/include/tao/pq/connection.hpp
@@ -6,6 +6,7 @@
 #define TAO_PQ_CONNECTION_HPP
 
 #include <chrono>
+#include <cstddef>
 #include <functional>
 #include <map>
 #include <memory>

diff --git a/include/tao/pq/connection_pool.hpp b/include/tao/pq/connection_pool.hpp
@@ -6,6 +6,7 @@
 #define TAO_PQ_CONNECTION_POOL_HPP
 
 #include <chrono>
+#include <functional>
 #include <memory>
 #include <optional>
 #include <string>

diff --git a/include/tao/pq/field.hpp b/include/tao/pq/field.hpp
@@ -11,7 +11,6 @@
 #include <utility>
 
 #include <tao/pq/null.hpp>
-#include <tao/pq/result_traits.hpp>
 
 namespace tao::pq
 {

diff --git a/include/tao/pq/internal/aggregate.hpp b/include/tao/pq/internal/aggregate.hpp
@@ -5,6 +5,7 @@
 #ifndef TAO_PQ_INTERNAL_AGGREGATE_HPP
 #define TAO_PQ_INTERNAL_AGGREGATE_HPP
 
+#include <cstddef>
 #include <tuple>
 #include <type_traits>
 #include <utility>

diff --git a/include/tao/pq/internal/from_chars.hpp b/include/tao/pq/internal/from_chars.hpp
@@ -8,8 +8,8 @@
 #include <charconv>
 #include <format>
 #include <stdexcept>
-#include <string>
 #include <string_view>
+#include <system_error>
 
 #include <tao/pq/internal/demangle.hpp>
 #include <tao/pq/internal/unreachable.hpp>

diff --git a/include/tao/pq/internal/gen.hpp b/include/tao/pq/internal/gen.hpp
@@ -5,6 +5,9 @@
 #ifndef TAO_PQ_INTERNAL_GEN_HPP
 #define TAO_PQ_INTERNAL_GEN_HPP
 
+#include <cstddef>
+#include <utility>
+
 #include <tao/pq/internal/exclusive_scan.hpp>
 
 namespace tao::pq::internal

diff --git a/include/tao/pq/internal/parameter_traits_helper.hpp b/include/tao/pq/internal/parameter_traits_helper.hpp
@@ -9,6 +9,7 @@
 #include <charconv>
 #include <cstddef>
 #include <string>
+#include <system_error>
 
 #include <tao/pq/oid.hpp>
 

diff --git a/include/tao/pq/internal/pool.hpp b/include/tao/pq/internal/pool.hpp
@@ -7,6 +7,7 @@
 
 #include <atomic>
 #include <cassert>
+#include <cstddef>
 #include <list>
 #include <memory>
 #include <mutex>

diff --git a/include/tao/pq/large_object.hpp b/include/tao/pq/large_object.hpp
@@ -5,10 +5,12 @@
 #ifndef TAO_PQ_LARGE_OBJECT_HPP
 #define TAO_PQ_LARGE_OBJECT_HPP
 
+#include <cstddef>
 #include <cstdint>
 #include <cstring>
 #include <ios>
 #include <memory>
+#include <string>
 
 #include <tao/pq/binary.hpp>
 #include <tao/pq/oid.hpp>

diff --git a/include/tao/pq/parameter_traits.hpp b/include/tao/pq/parameter_traits.hpp
@@ -6,7 +6,6 @@
 #define TAO_PQ_PARAMETER_TRAITS_HPP
 
 #include <cassert>
-#include <charconv>
 #include <cmath>
 #include <cstddef>
 #include <cstdio>

diff --git a/include/tao/pq/parameter_traits_array.hpp b/include/tao/pq/parameter_traits_array.hpp
@@ -10,7 +10,6 @@
 #include <list>
 #include <set>
 #include <string>
-#include <type_traits>
 #include <unordered_set>
 #include <vector>
 

diff --git a/include/tao/pq/parameter_traits_optional.hpp b/include/tao/pq/parameter_traits_optional.hpp
@@ -5,7 +5,9 @@
 #ifndef TAO_PQ_PARAMETER_TRAITS_OPTIONAL_HPP
 #define TAO_PQ_PARAMETER_TRAITS_OPTIONAL_HPP
 
+#include <cstddef>
 #include <optional>
+#include <string>
 #include <type_traits>
 #include <utility>
 

diff --git a/include/tao/pq/parameter_traits_pair.hpp b/include/tao/pq/parameter_traits_pair.hpp
@@ -5,6 +5,8 @@
 #ifndef TAO_PQ_PARAMETER_TRAITS_PAIR_HPP
 #define TAO_PQ_PARAMETER_TRAITS_PAIR_HPP
 
+#include <cstddef>
+#include <string>
 #include <type_traits>
 #include <utility>
 

diff --git a/include/tao/pq/parameter_traits_tuple.hpp b/include/tao/pq/parameter_traits_tuple.hpp
@@ -5,6 +5,8 @@
 #ifndef TAO_PQ_PARAMETER_TRAITS_TUPLE_HPP
 #define TAO_PQ_PARAMETER_TRAITS_TUPLE_HPP
 
+#include <cstddef>
+#include <string>
 #include <tuple>
 #include <type_traits>
 #include <utility>

diff --git a/include/tao/pq/result.hpp b/include/tao/pq/result.hpp
@@ -5,6 +5,7 @@
 #ifndef TAO_PQ_RESULT_HPP
 #define TAO_PQ_RESULT_HPP
 
+#include <cstddef>
 #include <cstdint>
 #include <format>
 #include <iterator>
@@ -140,7 +141,7 @@ namespace tao::pq
 
          friend void swap( const_iterator& lhs, const_iterator& rhs ) noexcept
          {
-            return swap( static_cast< row& >( lhs ), static_cast< row& >( rhs ) );
+            swap( static_cast< row& >( lhs ), static_cast< row& >( rhs ) );
          }
 
          [[nodiscard]] friend auto operator+( const const_iterator& lhs, const difference_type rhs ) noexcept -> const_iterator

diff --git a/include/tao/pq/result_traits.hpp b/include/tao/pq/result_traits.hpp
@@ -8,7 +8,6 @@
 #include <cstddef>
 #include <string>
 #include <string_view>
-#include <tuple>
 #include <type_traits>
 #include <utility>
 

diff --git a/include/tao/pq/result_traits_aggregate.hpp b/include/tao/pq/result_traits_aggregate.hpp
@@ -5,7 +5,13 @@
 #ifndef TAO_PQ_RESULT_TRAITS_AGGREGATE_HPP
 #define TAO_PQ_RESULT_TRAITS_AGGREGATE_HPP
 
+#include <cstddef>
+#include <tuple>
+#include <type_traits>
+#include <utility>
+
 #include <tao/pq/internal/aggregate.hpp>
+#include <tao/pq/internal/exclusive_scan.hpp>
 #include <tao/pq/is_aggregate.hpp>
 #include <tao/pq/result_traits.hpp>
 

diff --git a/include/tao/pq/result_traits_array.hpp b/include/tao/pq/result_traits_array.hpp
@@ -5,14 +5,11 @@
 #ifndef TAO_PQ_RESULT_TRAITS_ARRAY_HPP
 #define TAO_PQ_RESULT_TRAITS_ARRAY_HPP
 
-#include <array>
-#include <cstddef>
 #include <cstring>
 #include <list>
 #include <set>
 #include <stdexcept>
 #include <string>
-#include <type_traits>
 #include <unordered_set>
 #include <utility>
 #include <vector>
@@ -80,7 +77,7 @@ namespace tao::pq
             }
             else if( *value != '}' ) {
                if( const auto* end = std::strpbrk( value, ",;}" ) ) {
-                  std::string input( value, end );
+                  const std::string input( value, end );
                   if( input == "NULL" ) {
                      if constexpr( requires { result_traits< value_type >::null(); } ) {
                         container.push_back( result_traits< value_type >::null() );

diff --git a/include/tao/pq/result_traits_optional.hpp b/include/tao/pq/result_traits_optional.hpp
@@ -5,10 +5,10 @@
 #ifndef TAO_PQ_RESULT_TRAITS_OPTIONAL_HPP
 #define TAO_PQ_RESULT_TRAITS_OPTIONAL_HPP
 
+#include <cstddef>
 #include <optional>
 
 #include <tao/pq/result_traits.hpp>
-#include <tao/pq/row.hpp>
 
 template< typename T >
 struct tao::pq::result_traits< std::optional< T > >

diff --git a/include/tao/pq/result_traits_pair.hpp b/include/tao/pq/result_traits_pair.hpp
@@ -5,6 +5,7 @@
 #ifndef TAO_PQ_RESULT_TRAITS_PAIR_HPP
 #define TAO_PQ_RESULT_TRAITS_PAIR_HPP
 
+#include <cstddef>
 #include <type_traits>
 #include <utility>
 

diff --git a/include/tao/pq/result_traits_tuple.hpp b/include/tao/pq/result_traits_tuple.hpp
@@ -5,6 +5,7 @@
 #ifndef TAO_PQ_RESULT_TRAITS_TUPLE_HPP
 #define TAO_PQ_RESULT_TRAITS_TUPLE_HPP
 
+#include <cstddef>
 #include <tuple>
 #include <type_traits>
 #include <utility>

diff --git a/include/tao/pq/row.hpp b/include/tao/pq/row.hpp
@@ -136,7 +136,7 @@ namespace tao::pq
 
          friend void swap( const_iterator& lhs, const_iterator& rhs ) noexcept
          {
-            return swap( static_cast< field& >( lhs ), static_cast< field& >( rhs ) );
+            swap( static_cast< field& >( lhs ), static_cast< field& >( rhs ) );
          }
 
          [[nodiscard]] friend auto operator+( const const_iterator& lhs, const difference_type rhs ) noexcept -> const_iterator
@@ -249,7 +249,8 @@ namespace tao::pq
          return get< T >( 0 );
       }
 
-      template< typename T, typename = std::enable_if_t< is_aggregate_result< T > > >
+      template< typename T >
+         requires is_aggregate_result< T >
       [[nodiscard]] operator T() const
       {
          return as< T >();

diff --git a/include/tao/pq/table_field.hpp b/include/tao/pq/table_field.hpp
@@ -10,7 +10,6 @@
 #include <utility>
 
 #include <tao/pq/null.hpp>
-#include <tao/pq/result_traits.hpp>
 
 namespace tao::pq
 {

diff --git a/include/tao/pq/table_reader.hpp b/include/tao/pq/table_reader.hpp
@@ -6,12 +6,12 @@
 #define TAO_PQ_TABLE_READER_HPP
 
 #include <cassert>
+#include <cstddef>
 #include <cstdint>
 #include <list>
 #include <map>
 #include <memory>
 #include <set>
-#include <string>
 #include <string_view>
 #include <tuple>
 #include <unordered_map>
@@ -125,7 +125,7 @@ namespace tao::pq
 
          friend void swap( const_iterator& lhs, const_iterator& rhs ) noexcept
          {
-            return swap( static_cast< table_row& >( lhs ), static_cast< table_row& >( rhs ) );
+            swap( static_cast< table_row& >( lhs ), static_cast< table_row& >( rhs ) );
          }
 
          [[nodiscard]] friend auto operator==( const const_iterator& lhs, const const_iterator& rhs ) noexcept

diff --git a/include/tao/pq/table_row.hpp b/include/tao/pq/table_row.hpp
@@ -12,7 +12,6 @@
 #include <optional>
 #include <stdexcept>
 #include <tuple>
-#include <type_traits>
 #include <utility>
 
 #include <tao/pq/internal/demangle.hpp>
@@ -126,7 +125,7 @@ namespace tao::pq
 
          friend void swap( const_iterator& lhs, const_iterator& rhs ) noexcept
          {
-            return swap( static_cast< table_field& >( lhs ), static_cast< table_field& >( rhs ) );
+            swap( static_cast< table_field& >( lhs ), static_cast< table_field& >( rhs ) );
          }
 
          [[nodiscard]] friend auto operator+( const const_iterator& lhs, const difference_type rhs ) noexcept -> const_iterator

diff --git a/include/tao/pq/table_writer.hpp b/include/tao/pq/table_writer.hpp
@@ -5,6 +5,7 @@
 #ifndef TAO_PQ_TABLE_WRITER_HPP
 #define TAO_PQ_TABLE_WRITER_HPP
 
+#include <cstddef>
 #include <memory>
 #include <string>
 #include <string_view>

diff --git a/include/tao/pq/transaction.hpp b/include/tao/pq/transaction.hpp
@@ -9,7 +9,7 @@
 #include <cstddef>
 #include <cstdio>
 #include <memory>
-#include <string>
+#include <string_view>
 #if !defined( __cpp_pack_indexing ) && ( __cplusplus >= 202302L )
 #include <tuple>
 #endif