From 91b294e4a8f034f20a13c76c3112aabc67ce29e6 Mon Sep 17 00:00:00 2001 From: Ray Speth Date: Sun, 20 Mar 2022 22:48:39 -0400 Subject: [PATCH] WIP: mark constructors as explicit --- include/cantera/kinetics/Arrhenius.h | 3 ++- include/cantera/kinetics/BlowersMaselRate.h | 4 +++- include/cantera/kinetics/InterfaceRate.h | 4 ++-- 3 files changed, 7 insertions(+), 4 deletions(-) diff --git a/include/cantera/kinetics/Arrhenius.h b/include/cantera/kinetics/Arrhenius.h index 1bb85be980f..fa59c986119 100644 --- a/include/cantera/kinetics/Arrhenius.h +++ b/include/cantera/kinetics/Arrhenius.h @@ -70,7 +70,8 @@ class ArrheniusBase : public ReactionRate setRateParameters(rate, units, rate_units); } - ArrheniusBase(const AnyMap& node, const UnitStack& rate_units={}) : ArrheniusBase() + explicit ArrheniusBase(const AnyMap& node, const UnitStack& rate_units={}) + : ArrheniusBase() { setParameters(node, rate_units); } diff --git a/include/cantera/kinetics/BlowersMaselRate.h b/include/cantera/kinetics/BlowersMaselRate.h index ccf3886c6d6..c4ea6f4ad80 100644 --- a/include/cantera/kinetics/BlowersMaselRate.h +++ b/include/cantera/kinetics/BlowersMaselRate.h @@ -84,7 +84,9 @@ class BlowersMaselRate : public ArrheniusBase */ BlowersMaselRate(double A, double b, double Ea0, double w); - BlowersMaselRate(const AnyMap& node, const UnitStack& rate_units={}) : BlowersMaselRate() { + explicit BlowersMaselRate(const AnyMap& node, const UnitStack& rate_units={}) + : BlowersMaselRate() + { setParameters(node, rate_units); } diff --git a/include/cantera/kinetics/InterfaceRate.h b/include/cantera/kinetics/InterfaceRate.h index 4979f80762c..85e2eadaa95 100644 --- a/include/cantera/kinetics/InterfaceRate.h +++ b/include/cantera/kinetics/InterfaceRate.h @@ -359,7 +359,7 @@ class InterfaceRate : public RateType, public InterfaceRateBase using RateType::RateType; // inherit constructors //! Constructor based on AnyMap content - InterfaceRate(const AnyMap& node, const UnitStack& rate_units={}) { + explicit InterfaceRate(const AnyMap& node, const UnitStack& rate_units={}) { setParameters(node, rate_units); } @@ -457,7 +457,7 @@ class StickingRate : public RateType, public StickingCoverage using RateType::RateType; // inherit constructors //! Constructor based on AnyMap content - StickingRate(const AnyMap& node, const UnitStack& rate_units={}) { + explicit StickingRate(const AnyMap& node, const UnitStack& rate_units={}) { // sticking coefficients are dimensionless setParameters(node, Units(1.0)); }