mirror of
https://github.com/catchorg/Catch2.git
synced 2024-11-22 21:36:11 +01:00
Remove Config::openStream function
It has become completely vestigial, as it only ever passed-through the argument down to a different function, and being private member function, it didn't even introduce a useful compilation firewall.
This commit is contained in:
parent
46b3f7ee5f
commit
4b78157981
@ -79,10 +79,10 @@ namespace Catch {
|
|||||||
defaultOutputUsed = true;
|
defaultOutputUsed = true;
|
||||||
|
|
||||||
m_reporterStreams.push_back(
|
m_reporterStreams.push_back(
|
||||||
openStream( data.defaultOutputFilename ) );
|
makeStream( data.defaultOutputFilename ) );
|
||||||
} else {
|
} else {
|
||||||
m_reporterStreams.push_back(
|
m_reporterStreams.push_back(
|
||||||
openStream( *reporterSpec.outputFile() ) );
|
makeStream( *reporterSpec.outputFile() ) );
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
@ -139,8 +139,4 @@ namespace Catch {
|
|||||||
unsigned int Config::benchmarkResamples() const { return m_data.benchmarkResamples; }
|
unsigned int Config::benchmarkResamples() const { return m_data.benchmarkResamples; }
|
||||||
std::chrono::milliseconds Config::benchmarkWarmupTime() const { return std::chrono::milliseconds(m_data.benchmarkWarmupTime); }
|
std::chrono::milliseconds Config::benchmarkWarmupTime() const { return std::chrono::milliseconds(m_data.benchmarkWarmupTime); }
|
||||||
|
|
||||||
Detail::unique_ptr<IStream const> Config::openStream(std::string const& outputFileName) {
|
|
||||||
return Catch::makeStream(outputFileName);
|
|
||||||
}
|
|
||||||
|
|
||||||
} // end namespace Catch
|
} // end namespace Catch
|
||||||
|
@ -116,7 +116,6 @@ namespace Catch {
|
|||||||
std::chrono::milliseconds benchmarkWarmupTime() const override;
|
std::chrono::milliseconds benchmarkWarmupTime() const override;
|
||||||
|
|
||||||
private:
|
private:
|
||||||
Detail::unique_ptr<IStream const> openStream(std::string const& outputFileName);
|
|
||||||
ConfigData m_data;
|
ConfigData m_data;
|
||||||
|
|
||||||
std::vector<Detail::unique_ptr<IStream const>> m_reporterStreams;
|
std::vector<Detail::unique_ptr<IStream const>> m_reporterStreams;
|
||||||
|
Loading…
Reference in New Issue
Block a user