Skip to content

Commit

Permalink
Use parent constructor where applicable in reporters
Browse files Browse the repository at this point in the history
  • Loading branch information
horenmar committed Apr 12, 2022
1 parent d698776 commit 61f8031
Show file tree
Hide file tree
Showing 4 changed files with 4 additions and 12 deletions.
4 changes: 1 addition & 3 deletions src/catch2/reporters/catch_reporter_automake.hpp
Expand Up @@ -14,9 +14,7 @@ namespace Catch {

class AutomakeReporter final : public StreamingReporterBase {
public:
AutomakeReporter( ReporterConfig const& _config )
: StreamingReporterBase( _config )
{}
using StreamingReporterBase::StreamingReporterBase;

~AutomakeReporter() override;

Expand Down
3 changes: 1 addition & 2 deletions src/catch2/reporters/catch_reporter_cumulative_base.hpp
Expand Up @@ -89,8 +89,7 @@ namespace Catch {
using TestCaseNode = Node<TestCaseStats, SectionNode>;
using TestRunNode = Node<TestRunStats, TestCaseNode>;

CumulativeReporterBase( ReporterConfig const& _config ):
ReporterBase( _config ) {}
using ReporterBase::ReporterBase;
~CumulativeReporterBase() override;

void benchmarkPreparing( StringRef ) override {}
Expand Down
4 changes: 1 addition & 3 deletions src/catch2/reporters/catch_reporter_multi.hpp
Expand Up @@ -29,9 +29,7 @@ namespace Catch {
void updatePreferences(IEventListener const& reporterish);

public:
MultiReporter( IConfig const* config ):
IEventListener( config )
{}
using IEventListener::IEventListener;

void addListener( IEventListenerPtr&& listener );
void addReporter( IEventListenerPtr&& reporter );
Expand Down
5 changes: 1 addition & 4 deletions src/catch2/reporters/catch_reporter_streaming_base.hpp
Expand Up @@ -18,10 +18,7 @@ namespace Catch {

class StreamingReporterBase : public ReporterBase {
public:
StreamingReporterBase( ReporterConfig const& _config ):
ReporterBase( _config ) {}


using ReporterBase::ReporterBase;
~StreamingReporterBase() override;

void benchmarkPreparing( StringRef ) override {}
Expand Down

0 comments on commit 61f8031

Please sign in to comment.