From 3b297cf9b5aae20d152a844903a3260a91e85d65 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Martin=20Ho=C5=99e=C5=88ovsk=C3=BD?= Date: Sun, 2 Feb 2020 14:43:26 +0100 Subject: [PATCH] Outline Config destructor to avoid including superfluous header --- src/catch2/catch_config.cpp | 4 ++++ src/catch2/catch_config.hpp | 5 +---- 2 files changed, 5 insertions(+), 4 deletions(-) diff --git a/src/catch2/catch_config.cpp b/src/catch2/catch_config.cpp index 6d579af4..ea7ec3de 100644 --- a/src/catch2/catch_config.cpp +++ b/src/catch2/catch_config.cpp @@ -7,6 +7,7 @@ #include #include +#include #include namespace Catch { @@ -36,6 +37,9 @@ namespace Catch { m_testSpec = parser.testSpec(); } + Config::~Config() = default; + + std::string const& Config::getFilename() const { return m_data.outputFilename ; } diff --git a/src/catch2/catch_config.hpp b/src/catch2/catch_config.hpp index 85a7c84c..55fa5ed8 100644 --- a/src/catch2/catch_config.hpp +++ b/src/catch2/catch_config.hpp @@ -11,9 +11,6 @@ #include #include -// Libstdc++ doesn't like incomplete classes for unique_ptr -#include - #include #include #include @@ -73,7 +70,7 @@ namespace Catch { Config() = default; Config( ConfigData const& data ); - virtual ~Config() = default; + ~Config() override; // = default in the cpp file std::string const& getFilename() const;