From e36562caefea29fb754e07403331787a36c9cc50 Mon Sep 17 00:00:00 2001 From: Joachim Klein Date: Thu, 22 Mar 2018 18:32:02 +0100 Subject: [PATCH] (cleanup) IntervalIteration.h, Measures.h: make constructors explicit Single argument constructor may be mistakenly used as implicit conversion constructors, which does not make sense for these classes. --- prism/include/IntervalIteration.h | 2 +- prism/include/Measures.h | 4 ++-- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/prism/include/IntervalIteration.h b/prism/include/IntervalIteration.h index eb13008c..7f02e9e2 100644 --- a/prism/include/IntervalIteration.h +++ b/prism/include/IntervalIteration.h @@ -37,7 +37,7 @@ private: bool select_midpoint; public: - IntervalIteration(int flags) { + explicit IntervalIteration(int flags) { ensure_monotonic_from_below = (flags & 0x01); ensure_monotonic_from_above = (flags & 0x02); select_midpoint = (flags & 0x04); diff --git a/prism/include/Measures.h b/prism/include/Measures.h index 8cc62f83..91d6487c 100644 --- a/prism/include/Measures.h +++ b/prism/include/Measures.h @@ -45,7 +45,7 @@ private: public: /** Constructor, set relative flag */ - MeasureSupNorm(bool relative) : relative(relative) { + explicit MeasureSupNorm(bool relative) : relative(relative) { reset(); } @@ -117,7 +117,7 @@ private: public: /** Constructor, set relative flag */ - MeasureSupNormInterval(bool relative) : relative(relative) { + explicit MeasureSupNormInterval(bool relative) : relative(relative) { reset(); }