Don't cast nanoseconds to integers when writing output

This commit is contained in:
khyperia 2020-02-26 15:32:34 +01:00 committed by Martin Hořeňovský
parent 87b5bf77bc
commit 022b61ff92
2 changed files with 9 additions and 13 deletions

View File

@ -210,15 +210,11 @@ class Duration {
static const uint64_t s_nanosecondsInASecond = 1000 * s_nanosecondsInAMillisecond; static const uint64_t s_nanosecondsInASecond = 1000 * s_nanosecondsInAMillisecond;
static const uint64_t s_nanosecondsInAMinute = 60 * s_nanosecondsInASecond; static const uint64_t s_nanosecondsInAMinute = 60 * s_nanosecondsInASecond;
uint64_t m_inNanoseconds; double m_inNanoseconds;
Unit m_units; Unit m_units;
public: public:
explicit Duration(double inNanoseconds, Unit units = Unit::Auto) explicit Duration(double inNanoseconds, Unit units = Unit::Auto)
: Duration(static_cast<uint64_t>(inNanoseconds), units) {
}
explicit Duration(uint64_t inNanoseconds, Unit units = Unit::Auto)
: m_inNanoseconds(inNanoseconds), : m_inNanoseconds(inNanoseconds),
m_units(units) { m_units(units) {
if (m_units == Unit::Auto) { if (m_units == Unit::Auto) {
@ -247,7 +243,7 @@ public:
case Unit::Minutes: case Unit::Minutes:
return m_inNanoseconds / static_cast<double>(s_nanosecondsInAMinute); return m_inNanoseconds / static_cast<double>(s_nanosecondsInAMinute);
default: default:
return static_cast<double>(m_inNanoseconds); return m_inNanoseconds;
} }
} }
auto unitsAsString() const -> std::string { auto unitsAsString() const -> std::string {
@ -698,4 +694,4 @@ CATCH_REGISTER_REPORTER("console", ConsoleReporter)
#if defined(__clang__) #if defined(__clang__)
# pragma clang diagnostic pop # pragma clang diagnostic pop
#endif #endif

View File

@ -229,16 +229,16 @@ namespace Catch {
m_xml.writeAttribute("samples", info.samples) m_xml.writeAttribute("samples", info.samples)
.writeAttribute("resamples", info.resamples) .writeAttribute("resamples", info.resamples)
.writeAttribute("iterations", info.iterations) .writeAttribute("iterations", info.iterations)
.writeAttribute("clockResolution", static_cast<uint64_t>(info.clockResolution)) .writeAttribute("clockResolution", info.clockResolution)
.writeAttribute("estimatedDuration", static_cast<uint64_t>(info.estimatedDuration)) .writeAttribute("estimatedDuration", info.estimatedDuration)
.writeComment("All values in nano seconds"); .writeComment("All values in nano seconds");
} }
void XmlReporter::benchmarkEnded(BenchmarkStats<> const& benchmarkStats) { void XmlReporter::benchmarkEnded(BenchmarkStats<> const& benchmarkStats) {
m_xml.startElement("mean") m_xml.startElement("mean")
.writeAttribute("value", static_cast<uint64_t>(benchmarkStats.mean.point.count())) .writeAttribute("value", benchmarkStats.mean.point.count())
.writeAttribute("lowerBound", static_cast<uint64_t>(benchmarkStats.mean.lower_bound.count())) .writeAttribute("lowerBound", benchmarkStats.mean.lower_bound.count())
.writeAttribute("upperBound", static_cast<uint64_t>(benchmarkStats.mean.upper_bound.count())) .writeAttribute("upperBound", benchmarkStats.mean.upper_bound.count())
.writeAttribute("ci", benchmarkStats.mean.confidence_interval); .writeAttribute("ci", benchmarkStats.mean.confidence_interval);
m_xml.endElement(); m_xml.endElement();
m_xml.startElement("standardDeviation") m_xml.startElement("standardDeviation")