mirror of
https://github.com/catchorg/Catch2.git
synced 2024-12-23 03:43:28 +01:00
All of Approx's member functions now accept strong typedefs
Previously `Approx::operator()`, `Approx::epsilon`, `Approx::margin` and `Approx::scale` didn't. Closes #888
This commit is contained in:
parent
4cdb203ec3
commit
a34c053f0a
@ -40,16 +40,17 @@ namespace Detail {
|
|||||||
return Approx( 0 );
|
return Approx( 0 );
|
||||||
}
|
}
|
||||||
|
|
||||||
Approx operator()( double value ) {
|
#if defined(CATCH_CONFIG_CPP11_TYPE_TRAITS)
|
||||||
Approx approx( value );
|
|
||||||
|
template <typename T, typename = typename std::enable_if<std::is_constructible<double, T>::value>::type>
|
||||||
|
Approx operator()( T value ) {
|
||||||
|
Approx approx( static_cast<double>(value) );
|
||||||
approx.epsilon( m_epsilon );
|
approx.epsilon( m_epsilon );
|
||||||
approx.margin( m_margin );
|
approx.margin( m_margin );
|
||||||
approx.scale( m_scale );
|
approx.scale( m_scale );
|
||||||
return approx;
|
return approx;
|
||||||
}
|
}
|
||||||
|
|
||||||
#if defined(CATCH_CONFIG_CPP11_TYPE_TRAITS)
|
|
||||||
|
|
||||||
template <typename T, typename = typename std::enable_if<std::is_constructible<double, T>::value>::type>
|
template <typename T, typename = typename std::enable_if<std::is_constructible<double, T>::value>::type>
|
||||||
explicit Approx( T value ): Approx(static_cast<double>(value))
|
explicit Approx( T value ): Approx(static_cast<double>(value))
|
||||||
{}
|
{}
|
||||||
@ -100,7 +101,36 @@ namespace Detail {
|
|||||||
friend bool operator >= ( Approx const& lhs, T rhs ) {
|
friend bool operator >= ( Approx const& lhs, T rhs ) {
|
||||||
return lhs.m_value > double(rhs) || lhs == rhs;
|
return lhs.m_value > double(rhs) || lhs == rhs;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
template <typename T, typename = typename std::enable_if<std::is_constructible<double, T>::value>::type>
|
||||||
|
Approx& epsilon( T newEpsilon ) {
|
||||||
|
m_epsilon = double(newEpsilon);
|
||||||
|
return *this;
|
||||||
|
}
|
||||||
|
|
||||||
|
template <typename T, typename = typename std::enable_if<std::is_constructible<double, T>::value>::type>
|
||||||
|
Approx& margin( T newMargin ) {
|
||||||
|
m_margin = double(newMargin);
|
||||||
|
return *this;
|
||||||
|
}
|
||||||
|
|
||||||
|
template <typename T, typename = typename std::enable_if<std::is_constructible<double, T>::value>::type>
|
||||||
|
Approx& scale( T newScale ) {
|
||||||
|
m_scale = double(newScale);
|
||||||
|
return *this;
|
||||||
|
}
|
||||||
|
|
||||||
#else
|
#else
|
||||||
|
|
||||||
|
Approx operator()( double value ) {
|
||||||
|
Approx approx( value );
|
||||||
|
approx.epsilon( m_epsilon );
|
||||||
|
approx.margin( m_margin );
|
||||||
|
approx.scale( m_scale );
|
||||||
|
return approx;
|
||||||
|
}
|
||||||
|
|
||||||
|
|
||||||
friend bool operator == ( double lhs, Approx const& rhs ) {
|
friend bool operator == ( double lhs, Approx const& rhs ) {
|
||||||
// Thanks to Richard Harris for his help refining this formula
|
// Thanks to Richard Harris for his help refining this formula
|
||||||
bool relativeOK = std::fabs( lhs - rhs.m_value ) < rhs.m_epsilon * (rhs.m_scale + (std::max)( std::fabs(lhs), std::fabs(rhs.m_value) ) );
|
bool relativeOK = std::fabs( lhs - rhs.m_value ) < rhs.m_epsilon * (rhs.m_scale + (std::max)( std::fabs(lhs), std::fabs(rhs.m_value) ) );
|
||||||
@ -137,7 +167,6 @@ namespace Detail {
|
|||||||
friend bool operator >= ( Approx const& lhs, double rhs ) {
|
friend bool operator >= ( Approx const& lhs, double rhs ) {
|
||||||
return lhs.m_value > rhs || lhs == rhs;
|
return lhs.m_value > rhs || lhs == rhs;
|
||||||
}
|
}
|
||||||
#endif
|
|
||||||
|
|
||||||
Approx& epsilon( double newEpsilon ) {
|
Approx& epsilon( double newEpsilon ) {
|
||||||
m_epsilon = newEpsilon;
|
m_epsilon = newEpsilon;
|
||||||
@ -153,6 +182,7 @@ namespace Detail {
|
|||||||
m_scale = newScale;
|
m_scale = newScale;
|
||||||
return *this;
|
return *this;
|
||||||
}
|
}
|
||||||
|
#endif
|
||||||
|
|
||||||
std::string toString() const {
|
std::string toString() const {
|
||||||
std::ostringstream oss;
|
std::ostringstream oss;
|
||||||
|
Loading…
Reference in New Issue
Block a user