Rearrange the internal headers

.clang-format

@@ -10,3 +10,16 @@ FixNamespaceComments: true
 # IndentPPDirectives: AfterHash
 MacroBlockBegin: "^CONTINUABLE_BLOCK_.*_BEGIN$"
 MacroBlockEnd: "^CONTINUABLE_BLOCK_.*_END$"
+IncludeCategories:
+  - Regex:           '^<+[a-z_]+>'
+    Priority:        1
+  - Regex:           '^<experimental/+[a-z_]+>'
+    Priority:        2
+  - Regex:           '^<(gtest|function2)/.*\.(h|hpp)>'
+    Priority:        3
+  - Regex:           '^<continuable/.*\.hpp>'
+    Priority:        4
+  - Regex:           '^<.*'
+    Priority:        5
+  - Regex:           '.*'
+    Priority:        6

include/continuable/continuable-base.hpp

@@ -35,20 +35,19 @@
 #include <cstddef>
 #include <type_traits>
 #include <utility>
-
-#include <continuable/detail/base.hpp>
-#include <continuable/detail/connection-all.hpp>
-#include <continuable/detail/connection-any.hpp>
-#include <continuable/detail/connection-seq.hpp>
-#include <continuable/detail/connection.hpp>
+#include <continuable/detail/connection/connection-all.hpp>
+#include <continuable/detail/connection/connection-any.hpp>
+#include <continuable/detail/connection/connection-seq.hpp>
+#include <continuable/detail/connection/connection.hpp>
+#include <continuable/detail/core/base.hpp>
+#include <continuable/detail/core/types.hpp>
 #include <continuable/detail/features.hpp>
-#include <continuable/detail/traits.hpp>
-#include <continuable/detail/types.hpp>
-#include <continuable/detail/util.hpp>
+#include <continuable/detail/utility/traits.hpp>
+#include <continuable/detail/utility/util.hpp>
 
 #ifdef CONTINUABLE_HAS_EXPERIMENTAL_COROUTINE
-#include <continuable/detail/awaiting.hpp>
 #include <experimental/coroutine>
+#include <continuable/detail/other/coroutines.hpp>
 #endif // CONTINUABLE_HAS_EXPERIMENTAL_COROUTINE
 
 namespace cti {

include/continuable/continuable-connections.hpp

@@ -35,12 +35,11 @@
 #include <memory>
 #include <utility>
 #include <vector>
-
-#include <continuable/detail/connection-all.hpp>
-#include <continuable/detail/connection-any.hpp>
-#include <continuable/detail/connection-seq.hpp>
-#include <continuable/detail/connection.hpp>
-#include <continuable/detail/range.hpp>
+#include <continuable/detail/connection/connection-all.hpp>
+#include <continuable/detail/connection/connection-any.hpp>
+#include <continuable/detail/connection/connection-seq.hpp>
+#include <continuable/detail/connection/connection.hpp>
+#include <continuable/detail/traversal/range.hpp>
 
 namespace cti {
 /// \defgroup Connections Connections

include/continuable/continuable-coroutine.hpp

@@ -32,15 +32,15 @@
 
 #include <continuable/continuable-base.hpp>
 #include <continuable/continuable-types.hpp>
+#include <continuable/detail/core/types.hpp>
 #include <continuable/detail/features.hpp>
-#include <continuable/detail/types.hpp>
 
 #if defined(CONTINUABLE_HAS_EXCEPTIONS)
 #include <exception>
 #endif // CONTINUABLE_HAS_EXCEPTIONS
 
 #ifdef CONTINUABLE_HAS_EXPERIMENTAL_COROUTINE
-#include <continuable/detail/awaiting.hpp>
+#include <continuable/detail/other/coroutines.hpp>
 #endif // CONTINUABLE_HAS_EXPERIMENTAL_COROUTINE
 
 /// \cond false

include/continuable/continuable-promise-base.hpp

@@ -33,10 +33,9 @@
 
 #include <type_traits>
 #include <utility>
-
-#include <continuable/detail/hints.hpp>
-#include <continuable/detail/types.hpp>
-#include <continuable/detail/util.hpp>
+#include <continuable/detail/core/hints.hpp>
+#include <continuable/detail/core/types.hpp>
+#include <continuable/detail/utility/util.hpp>
 
 namespace cti {
 /// \defgroup Base Base

include/continuable/continuable-promisify.hpp

@@ -33,8 +33,7 @@
 
 #include <type_traits>
 #include <utility>
-
-#include <continuable/detail/promisify.hpp>
+#include <continuable/detail/other/promisify.hpp>
 
 namespace cti {
 /// \defgroup Promisify Promisify

include/continuable/continuable-testing.hpp

@@ -31,8 +31,8 @@
 #ifndef CONTINUABLE_TESTING_HPP_INCLUDED
 #define CONTINUABLE_TESTING_HPP_INCLUDED
 
-#include <continuable/detail/testing.hpp>
-#include <continuable/detail/traits.hpp>
+#include <continuable/detail/other/testing.hpp>
+#include <continuable/detail/utility/traits.hpp>
 
 /// \defgroup Testing Testing
 /// provides macro shortcuts for testing asynchronous continuations through

include/continuable/continuable-trait.hpp

@@ -32,11 +32,10 @@
 #define CONTINUABLE_TRAIT_HPP_INCLUDED
 
 #include <cstddef>
-
 #include <continuable/continuable-base.hpp>
 #include <continuable/continuable-promise-base.hpp>
-#include <continuable/detail/hints.hpp>
-#include <continuable/detail/types.hpp>
+#include <continuable/detail/core/hints.hpp>
+#include <continuable/detail/core/types.hpp>
 
 namespace cti {
 /// \defgroup Types Types

include/continuable/continuable-transforms.hpp

@@ -31,8 +31,8 @@
 #ifndef CONTINUABLE_TRANSFORMS_HPP_INCLUDED
 #define CONTINUABLE_TRANSFORMS_HPP_INCLUDED
 
-#include <continuable/detail/transforms.hpp>
-#include <continuable/detail/types.hpp>
+#include <continuable/detail/core/types.hpp>
+#include <continuable/detail/other/transforms.hpp>
 
 namespace cti {
 /// \defgroup Transforms Transforms

include/continuable/continuable-traverse-async.hpp

@@ -32,8 +32,7 @@
 #define CONTINUABLE_TRAVERSE_ASYNC_HPP_INCLUDED
 
 #include <utility>
-
-#include <continuable/detail/traverse-async.hpp>
+#include <continuable/detail/traversal/traverse-async.hpp>
 
 namespace cti {
 /// \defgroup Traversal Traversal

include/continuable/continuable-traverse.hpp

@@ -34,8 +34,7 @@
 #include <tuple>
 #include <type_traits>
 #include <utility>
-
-#include <continuable/detail/traverse.hpp>
+#include <continuable/detail/traversal/traverse.hpp>
 
 namespace cti {
 /// \defgroup Traversal Traversal

include/continuable/continuable-types.hpp

@@ -32,9 +32,7 @@
 #define CONTINUABLE_TYPES_HPP_INCLUDED
 
 #include <cstddef>
-
 #include <function2/function2.hpp>
-
 #include <continuable/continuable-trait.hpp>
 
 namespace cti {

include/continuable/detail/connection/connection-aggregated.hpp

@@ -35,11 +35,10 @@
 #include <tuple>
 #include <type_traits>
 #include <utility>
-
 #include <continuable/continuable-traverse.hpp>
-#include <continuable/detail/base.hpp>
-#include <continuable/detail/flat-variant.hpp>
-#include <continuable/detail/traits.hpp>
+#include <continuable/detail/core/base.hpp>
+#include <continuable/detail/utility/flat-variant.hpp>
+#include <continuable/detail/utility/traits.hpp>
 
 namespace cti {
 namespace detail {
@@ -211,7 +210,7 @@ constexpr auto finalize_impl(traits::identity<std::tuple<Args...>>,
                              Callback&& callback, Data&& data) {
   // Call the final callback with the cleaned result
   return traits::unpack(std::forward<Callback>(callback),
-                       unbox_continuables(std::forward<Data>(data)));
+                        unbox_continuables(std::forward<Data>(data)));
 }
 
 struct hint_mapper {

include/continuable/detail/connection/connection-all.hpp

@@ -37,13 +37,12 @@
 #include <tuple>
 #include <type_traits>
 #include <utility>
-
-#include <continuable/detail/base.hpp>
-#include <continuable/detail/connection-aggregated.hpp>
-#include <continuable/detail/connection.hpp>
-#include <continuable/detail/hints.hpp>
-#include <continuable/detail/traits.hpp>
-#include <continuable/detail/types.hpp>
+#include <continuable/detail/connection/connection-aggregated.hpp>
+#include <continuable/detail/connection/connection.hpp>
+#include <continuable/detail/core/base.hpp>
+#include <continuable/detail/core/hints.hpp>
+#include <continuable/detail/core/types.hpp>
+#include <continuable/detail/utility/traits.hpp>
 
 namespace cti {
 namespace detail {

include/continuable/detail/connection/connection-any.hpp

@@ -37,14 +37,13 @@
 #include <tuple>
 #include <type_traits>
 #include <utility>
-
 #include <continuable/continuable-promise-base.hpp>
 #include <continuable/continuable-traverse.hpp>
-#include <continuable/detail/base.hpp>
-#include <continuable/detail/container-category.hpp>
-#include <continuable/detail/hints.hpp>
-#include <continuable/detail/traits.hpp>
-#include <continuable/detail/types.hpp>
+#include <continuable/detail/core/base.hpp>
+#include <continuable/detail/core/hints.hpp>
+#include <continuable/detail/core/types.hpp>
+#include <continuable/detail/traversal/container-category.hpp>
+#include <continuable/detail/utility/traits.hpp>
 
 namespace cti {
 namespace detail {

include/continuable/detail/connection/connection-seq.hpp

@@ -36,12 +36,11 @@
 #include <tuple>
 #include <type_traits>
 #include <utility>
-
 #include <continuable/continuable-traverse-async.hpp>
-#include <continuable/detail/base.hpp>
-#include <continuable/detail/connection-aggregated.hpp>
-#include <continuable/detail/traits.hpp>
-#include <continuable/detail/util.hpp>
+#include <continuable/detail/connection/connection-aggregated.hpp>
+#include <continuable/detail/core/base.hpp>
+#include <continuable/detail/utility/traits.hpp>
+#include <continuable/detail/utility/util.hpp>
 
 namespace cti {
 namespace detail {

include/continuable/detail/connection/connection.hpp

@@ -35,12 +35,11 @@
 #include <tuple>
 #include <type_traits>
 #include <utility>
-
 #include <continuable/continuable-traverse.hpp>
-#include <continuable/detail/base.hpp>
-#include <continuable/detail/traits.hpp>
-#include <continuable/detail/types.hpp>
-#include <continuable/detail/util.hpp>
+#include <continuable/detail/core/base.hpp>
+#include <continuable/detail/core/types.hpp>
+#include <continuable/detail/utility/traits.hpp>
+#include <continuable/detail/utility/util.hpp>
 
 namespace cti {
 namespace detail {

include/continuable/detail/core/base.hpp

@@ -34,12 +34,11 @@
 #include <tuple>
 #include <type_traits>
 #include <utility>
-
+#include <continuable/detail/core/hints.hpp>
+#include <continuable/detail/core/types.hpp>
 #include <continuable/detail/features.hpp>
-#include <continuable/detail/hints.hpp>
-#include <continuable/detail/traits.hpp>
-#include <continuable/detail/types.hpp>
-#include <continuable/detail/util.hpp>
+#include <continuable/detail/utility/traits.hpp>
+#include <continuable/detail/utility/util.hpp>
 
 #if defined(CONTINUABLE_HAS_EXCEPTIONS)
 #include <exception>

include/continuable/detail/core/hints.hpp

@@ -32,9 +32,8 @@
 #define CONTINUABLE_DETAIL_HINTS_HPP_INCLUDED
 
 #include <type_traits>
-
-#include <continuable/detail/traits.hpp>
-#include <continuable/detail/types.hpp>
+#include <continuable/detail/core/types.hpp>
+#include <continuable/detail/utility/traits.hpp>
 
 namespace cti {
 namespace detail {

include/continuable/detail/core/types.hpp

@@ -32,7 +32,6 @@
 #define CONTINUABLE_DETAIL_TYPES_HPP_INCLUDED
 
 #include <utility>
-
 #include <continuable/detail/features.hpp>
 
 #ifndef CONTINUABLE_WITH_CUSTOM_ERROR_TYPE

include/continuable/detail/other/coroutines.hpp

@@ -33,15 +33,14 @@
 #define CONTINUABLE_DETAIL_AWAITING_HPP_INCLUDED
 
 #include <cassert>
-#include <experimental/coroutine>
 #include <tuple>
-
-#include <continuable/detail/expected.hpp>
+#include <experimental/coroutine>
+#include <continuable/detail/core/hints.hpp>
+#include <continuable/detail/core/types.hpp>
 #include <continuable/detail/features.hpp>
-#include <continuable/detail/hints.hpp>
-#include <continuable/detail/traits.hpp>
-#include <continuable/detail/types.hpp>
-#include <continuable/detail/util.hpp>
+#include <continuable/detail/utility/expected.hpp>
+#include <continuable/detail/utility/traits.hpp>
+#include <continuable/detail/utility/util.hpp>
 
 #if defined(CONTINUABLE_HAS_EXCEPTIONS)
 #include <exception>
@@ -172,8 +171,7 @@ struct promise_resolver_base<promise_type<Promise, Args...>> {
 };
 
 template <typename Promise, typename... Args>
-struct promise_type
-    : promise_resolver_base<promise_type<Promise, Args...>> {
+struct promise_type : promise_resolver_base<promise_type<Promise, Args...>> {
 
   coroutine_handle<> handle_;
   Promise promise_;

include/continuable/detail/other/promisify.hpp

@@ -32,15 +32,15 @@
 #define CONTINUABLE_DETAIL_PROMISIFY_HPP_INCLUDED
 
 #include <type_traits>
+#include <continuable/continuable-base.hpp>
+#include <continuable/detail/features.hpp>
+#include <continuable/detail/utility/traits.hpp>
+#include <continuable/detail/utility/util.hpp>
 
 #if defined(CONTINUABLE_HAS_EXCEPTIONS)
 #include <exception>
 #endif // CONTINUABLE_HAS_EXCEPTIONS
 
-#include <continuable/continuable-base.hpp>
-#include <continuable/detail/traits.hpp>
-#include <continuable/detail/util.hpp>
-
 namespace cti {
 namespace detail {
 namespace convert {

include/continuable/detail/other/testing.hpp

@@ -33,13 +33,11 @@
 
 #include <type_traits>
 #include <utility>
-
 #include <gtest/gtest.h>
-
+#include <continuable/detail/core/types.hpp>
 #include <continuable/detail/features.hpp>
-#include <continuable/detail/traits.hpp>
-#include <continuable/detail/types.hpp>
-#include <continuable/detail/util.hpp>
+#include <continuable/detail/utility/traits.hpp>
+#include <continuable/detail/utility/util.hpp>
 
 namespace cti {
 namespace detail {

include/continuable/detail/other/transforms.hpp

@@ -32,12 +32,11 @@
 #define CONTINUABLE_DETAIL_TRANSFORMS_HPP_INCLUDED
 
 #include <future>
-
-#include <continuable/detail/base.hpp>
+#include <continuable/detail/core/base.hpp>
+#include <continuable/detail/core/hints.hpp>
+#include <continuable/detail/core/types.hpp>
 #include <continuable/detail/features.hpp>
-#include <continuable/detail/hints.hpp>
-#include <continuable/detail/types.hpp>
-#include <continuable/detail/util.hpp>
+#include <continuable/detail/utility/util.hpp>
 
 namespace cti {
 namespace detail {

include/continuable/detail/traversal/container-category.hpp

@@ -33,8 +33,7 @@
 
 #include <tuple>
 #include <type_traits>
-
-#include <continuable/detail/traits.hpp>
+#include <continuable/detail/utility/traits.hpp>
 
 namespace cti {
 namespace detail {

include/continuable/detail/traversal/range.hpp

@@ -35,8 +35,7 @@
 #include <type_traits>
 #include <utility>
 #include <vector>
-
-#include <continuable/detail/traits.hpp>
+#include <continuable/detail/utility/traits.hpp>
 
 namespace cti {
 namespace detail {

include/continuable/detail/traversal/traverse-async.hpp

@@ -39,9 +39,8 @@
 #include <tuple>
 #include <type_traits>
 #include <utility>
-
-#include <continuable/detail/container-category.hpp>
-#include <continuable/detail/traits.hpp>
+#include <continuable/detail/traversal/container-category.hpp>
+#include <continuable/detail/utility/traits.hpp>
 
 namespace cti {
 namespace detail {

include/continuable/detail/traversal/traverse.hpp

@@ -37,9 +37,8 @@
 #include <tuple>
 #include <type_traits>
 #include <utility>
-
-#include <continuable/detail/container-category.hpp>
-#include <continuable/detail/traits.hpp>
+#include <continuable/detail/traversal/container-category.hpp>
+#include <continuable/detail/utility/traits.hpp>
 
 namespace cti {
 namespace detail {
@@ -193,9 +192,9 @@ template <typename C, typename... T>
 constexpr auto apply_spread_impl(std::true_type, C&& callable, T&&... args)
     -> decltype(
         traits::unpack(std::forward<C>(callable),
-                      std::tuple_cat(undecorate(std::forward<T>(args))...))) {
+                       std::tuple_cat(undecorate(std::forward<T>(args))...))) {
   return traits::unpack(std::forward<C>(callable),
-                       std::tuple_cat(undecorate(std::forward<T>(args))...));
+                        std::tuple_cat(undecorate(std::forward<T>(args))...));
 }
 
 /// Use the linear instantiation for variadic packs which don't

include/continuable/detail/utility/expected.hpp

@@ -33,10 +33,9 @@
 
 #include <type_traits>
 #include <utility>
-
-#include <continuable/detail/flat-variant.hpp>
-#include <continuable/detail/hints.hpp>
-#include <continuable/detail/types.hpp>
+#include <continuable/detail/core/hints.hpp>
+#include <continuable/detail/core/types.hpp>
+#include <continuable/detail/utility/flat-variant.hpp>
 
 namespace cti {
 namespace detail {

include/continuable/detail/utility/flat-variant.hpp

@@ -38,8 +38,7 @@
 #include <memory>
 #include <type_traits>
 #include <utility>
-
-#include <continuable/detail/traits.hpp>
+#include <continuable/detail/utility/traits.hpp>
 
 namespace cti {
 namespace detail {

include/continuable/detail/utility/traits.hpp

@@ -35,7 +35,6 @@
 #include <tuple>
 #include <type_traits>
 #include <utility>
-
 #include <continuable/detail/features.hpp>
 
 namespace cti {

include/continuable/detail/utility/util.hpp

@@ -35,9 +35,8 @@
 #include <tuple>
 #include <type_traits>
 #include <utility>
-
 #include <continuable/detail/features.hpp>
-#include <continuable/detail/traits.hpp>
+#include <continuable/detail/utility/traits.hpp>
 
 namespace cti {
 namespace detail {

test/playground/CMakeLists.txt

@@ -1,7 +1,12 @@
 set(INCLUDE_DIR ${CMAKE_SOURCE_DIR}/include/continuable)
 
-file(GLOB LIB_SOURCES ${INCLUDE_DIR}/*.hpp)
-file(GLOB_RECURSE LIB_SOURCES_DETAIL ${INCLUDE_DIR}/detail/*.hpp)
+file(GLOB LIB_SOURCES CONFIGURE_DEPENDS ${INCLUDE_DIR}/*.hpp)
+file(GLOB LIB_SOURCES_DETAIL CONFIGURE_DEPENDS ${INCLUDE_DIR}/detail/*.hpp)
+file(GLOB LIB_SOURCES_DETAIL_CONNECTIONS CONFIGURE_DEPENDS ${INCLUDE_DIR}/detail/connection/*.hpp)
+file(GLOB LIB_SOURCES_DETAIL_CORE CONFIGURE_DEPENDS ${INCLUDE_DIR}/detail/core/*.hpp)
+file(GLOB LIB_SOURCES_DETAIL_OTHER CONFIGURE_DEPENDS ${INCLUDE_DIR}/detail/other/*.hpp)
+file(GLOB LIB_SOURCES_DETAIL_TRAVERSAL CONFIGURE_DEPENDS ${INCLUDE_DIR}/detail/traversal/*.hpp)
+file(GLOB LIB_SOURCES_DETAIL_UTILITY CONFIGURE_DEPENDS ${INCLUDE_DIR}/detail/utility/*.hpp)
 
 set(TEST
   ${CMAKE_CURRENT_LIST_DIR}/test-playground.cpp)
@@ -9,10 +14,20 @@ set(TEST
 add_executable(test-playground
   ${LIB_SOURCES}
   ${LIB_SOURCES_DETAIL}
+  ${LIB_SOURCES_DETAIL_CONNECTIONS}
+  ${LIB_SOURCES_DETAIL_CORE}
+  ${LIB_SOURCES_DETAIL_OTHER}
+  ${LIB_SOURCES_DETAIL_TRAVERSAL}
+  ${LIB_SOURCES_DETAIL_UTILITY}
   ${TEST})
 
 source_group(continuable FILES ${LIB_SOURCES})
 source_group(continuable\\detail FILES ${LIB_SOURCES_DETAIL})
+source_group(continuable\\detail\\connections FILES ${LIB_SOURCES_DETAIL_CONNECTIONS})
+source_group(continuable\\detail\\core FILES ${LIB_SOURCES_DETAIL_CORE})
+source_group(continuable\\detail\\other FILES ${LIB_SOURCES_DETAIL_OTHER})
+source_group(continuable\\detail\\traversal FILES ${LIB_SOURCES_DETAIL_TRAVERSAL})
+source_group(continuable\\detail\\utility FILES ${LIB_SOURCES_DETAIL_UTILITY})
 source_group(test FILES ${TEST})
 
 target_link_libraries(test-playground