Some IWYU include cleanups

src/catch2/benchmark/detail/catch_stats.cpp

@@ -12,6 +12,7 @@
 #include <catch2/internal/catch_compiler_capabilities.hpp>
 
 #include <cassert>
+#include <cstddef>
 #include <iterator>
 #include <random>
 

src/catch2/catch_approx.cpp

@@ -7,6 +7,7 @@
 // SPDX-License-Identifier: BSL-1.0
 #include <catch2/catch_approx.hpp>
 #include <catch2/internal/catch_enforce.hpp>
+#include <catch2/internal/catch_stream.hpp>
 
 #include <cmath>
 #include <limits>

src/catch2/catch_config.cpp

@@ -14,8 +14,6 @@
 #include <catch2/internal/catch_test_spec_parser.hpp>
 #include <catch2/interfaces/catch_interfaces_tag_alias_registry.hpp>
 
-#include <ostream>
-
 namespace Catch {
 
     bool operator==( ProcessedReporterSpec const& lhs,

src/catch2/catch_config.hpp

@@ -9,16 +9,17 @@
 #define CATCH_CONFIG_HPP_INCLUDED
 
 #include <catch2/catch_test_spec.hpp>
-#include <catch2/catch_user_config.hpp>
 #include <catch2/interfaces/catch_interfaces_config.hpp>
 #include <catch2/internal/catch_unique_ptr.hpp>
 #include <catch2/internal/catch_optional.hpp>
+#include <catch2/internal/catch_stringref.hpp>
 #include <catch2/internal/catch_random_seed_generation.hpp>
 #include <catch2/internal/catch_reporter_spec_parser.hpp>
 
-#include <iosfwd>
+#include <chrono>
-#include <vector>
+#include <map>
 #include <string>
+#include <vector>
 
 namespace Catch {
 

src/catch2/catch_test_case_info.cpp

@@ -7,8 +7,6 @@
 // SPDX-License-Identifier: BSL-1.0
 #include <catch2/catch_test_case_info.hpp>
 #include <catch2/internal/catch_enforce.hpp>
-#include <catch2/catch_test_spec.hpp>
-#include <catch2/interfaces/catch_interfaces_testcase.hpp>
 #include <catch2/internal/catch_string_manip.hpp>
 #include <catch2/internal/catch_case_insensitive_comparisons.hpp>
 

src/catch2/catch_totals.hpp

@@ -8,7 +8,6 @@
 #ifndef CATCH_TOTALS_HPP_INCLUDED
 #define CATCH_TOTALS_HPP_INCLUDED
 
-#include <cstddef>
 #include <cstdint>
 
 namespace Catch {

src/catch2/internal/catch_assertion_handler.hpp

@@ -13,11 +13,11 @@
 #include <catch2/interfaces/catch_interfaces_capture.hpp>
 #include <catch2/internal/catch_lazy_expr.hpp>
 
+#include <string>
+
 namespace Catch {
 
-    struct AssertionResultData;
     class IResultCapture;
-    class RunContext;
 
     struct AssertionReaction {
         bool shouldDebugBreak = false;

src/catch2/internal/catch_case_insensitive_comparisons.hpp

@@ -10,8 +10,6 @@
 
 #include <catch2/internal/catch_stringref.hpp>
 
-#include <string>
-
 namespace Catch {
     namespace Detail {
         //! Provides case-insensitive `op<` semantics when called

src/catch2/internal/catch_clara.cpp

@@ -5,13 +5,16 @@
 //        https://www.boost.org/LICENSE_1_0.txt)
 
 // SPDX-License-Identifier: BSL-1.0
-#include <algorithm>
+
 #include <catch2/internal/catch_clara.hpp>
 #include <catch2/internal/catch_console_width.hpp>
 #include <catch2/internal/catch_platform.hpp>
 #include <catch2/internal/catch_string_manip.hpp>
 #include <catch2/internal/catch_textflow.hpp>
 
+#include <algorithm>
+#include <ostream>
+
 namespace {
     bool isOptPrefix( char c ) {
         return c == '-'

src/catch2/internal/catch_clara.hpp

@@ -34,7 +34,6 @@
 #include <catch2/internal/catch_void_type.hpp>
 
 #include <cassert>
-#include <cctype>
 #include <memory>
 #include <ostream>
 #include <sstream>

src/catch2/internal/catch_commandline.cpp

@@ -12,11 +12,9 @@
 #include <catch2/internal/catch_string_manip.hpp>
 #include <catch2/interfaces/catch_interfaces_registry_hub.hpp>
 #include <catch2/interfaces/catch_interfaces_reporter_registry.hpp>
-#include <catch2/interfaces/catch_interfaces_reporter.hpp>
 #include <catch2/internal/catch_console_colour.hpp>
 #include <catch2/internal/catch_reporter_spec_parser.hpp>
 
-#include <algorithm>
 #include <fstream>
 #include <string>
 

src/catch2/internal/catch_console_colour.cpp

@@ -16,13 +16,16 @@
 #include <catch2/internal/catch_errno_guard.hpp>
 #include <catch2/interfaces/catch_interfaces_config.hpp>
 #include <catch2/internal/catch_stream.hpp>
+#include <catch2/internal/catch_move_and_forward.hpp>
 #include <catch2/internal/catch_context.hpp>
 #include <catch2/internal/catch_platform.hpp>
 #include <catch2/internal/catch_debugger.hpp>
 #include <catch2/internal/catch_windows_h_proxy.hpp>
 #include <catch2/internal/catch_compiler_capabilities.hpp>
 
+#include <cassert>
 #include <ostream>
+#include <utility>
 
 namespace Catch {
 

src/catch2/internal/catch_console_colour.hpp

@@ -16,7 +16,6 @@
 namespace Catch {
 
     enum class ColourMode : std::uint8_t;
-    class IConfig;
     class IStream;
 
     struct Colour {

src/catch2/internal/catch_debug_console.cpp

@@ -11,6 +11,8 @@
 #include <catch2/internal/catch_platform.hpp>
 #include <catch2/internal/catch_windows_h_proxy.hpp>
 
+#include <iosfwd>
+
 #if defined(CATCH_CONFIG_ANDROID_LOGWRITE)
 #include <android/log.h>
 

src/catch2/internal/catch_reporter_spec_parser.cpp

@@ -12,7 +12,6 @@
 #include <catch2/internal/catch_move_and_forward.hpp>
 
 #include <algorithm>
-#include <ostream>
 
 namespace Catch {
 

src/catch2/internal/catch_tag_alias_registry.hpp

@@ -15,6 +15,7 @@
 #include <string>
 
 namespace Catch {
+    struct SourceLineInfo;
 
     class TagAliasRegistry : public ITagAliasRegistry {
     public:

src/catch2/internal/catch_xmlwriter.hpp

@@ -11,6 +11,7 @@
 #include <catch2/internal/catch_stream.hpp>
 #include <catch2/internal/catch_stringref.hpp>
 
+#include <iosfwd>
 #include <vector>
 
 namespace Catch {

src/catch2/reporters/catch_reporter_automake.hpp

@@ -10,6 +10,8 @@
 
 #include <catch2/reporters/catch_reporter_streaming_base.hpp>
 
+#include <string>
+
 namespace Catch {
 
     class AutomakeReporter final : public StreamingReporterBase {

src/catch2/reporters/catch_reporter_cumulative_base.cpp

@@ -7,6 +7,8 @@
 // SPDX-License-Identifier: BSL-1.0
 #include <catch2/reporters/catch_reporter_cumulative_base.hpp>
 
+#include <catch2/internal/catch_move_and_forward.hpp>
+
 #include <algorithm>
 #include <cassert>
 

src/catch2/reporters/catch_reporter_cumulative_base.hpp

@@ -12,7 +12,6 @@
 #include <catch2/internal/catch_unique_ptr.hpp>
 #include <catch2/internal/catch_optional.hpp>
 
-#include <iosfwd>
 #include <string>
 #include <vector>
 

src/catch2/reporters/catch_reporter_multi.cpp

@@ -11,8 +11,6 @@
 #include <catch2/internal/catch_move_and_forward.hpp>
 #include <catch2/internal/catch_stream.hpp>
 
-#include <cassert>
-
 namespace Catch {
     void MultiReporter::updatePreferences(IEventListener const& reporterish) {
         m_preferences.shouldRedirectStdOut |=

src/catch2/reporters/catch_reporter_streaming_base.hpp

@@ -10,8 +10,6 @@
 
 #include <catch2/reporters/catch_reporter_common_base.hpp>
 
-#include <iosfwd>
-#include <string>
 #include <vector>
 
 namespace Catch {

src/catch2/reporters/catch_reporter_tap.cpp

@@ -11,6 +11,7 @@
 #include <catch2/catch_test_case_info.hpp>
 
 #include <algorithm>
+#include <iterator>
 #include <ostream>
 
 namespace Catch {

src/catch2/reporters/catch_reporter_teamcity.cpp

@@ -14,6 +14,7 @@
 #include <catch2/catch_test_case_info.hpp>
 
 #include <cassert>
+#include <ostream>
 
 namespace Catch {
 

tests/ExtraTests/X05-DeferredStaticChecks.cpp

@@ -12,7 +12,6 @@
  */
 
 #include <catch2/catch_test_macros.hpp>
-#include <iostream>
 
 TEST_CASE("Deferred static checks") {
     STATIC_CHECK(1 == 2);