diff options
author | Volker Hilsheimer <volker.hilsheimer@qt.io> | 2020-10-26 16:34:20 +0100 |
---|---|---|
committer | Volker Hilsheimer <volker.hilsheimer@qt.io> | 2020-10-26 20:10:49 +0100 |
commit | feda3e7673137c3f6a9f3561276b6d21447fd881 (patch) | |
tree | 43724a514793a2a91926182a6b8776ffa2e3cbbf /src | |
parent | 525372c56771a548ac2842ee860730c6ad1e5577 (diff) |
Make QRect(F) and QMargins comparison operators hidden friends
Hide them from ADL to reduce noise. QRect operators were already
implemented as hidden friends, but a left-over declaration un-hid them
again.
Use the opportunity to simplify QMargins::operator!= by just inverting
operator==. No need to spell things out, the compiler can do that for us.
Change-Id: I55722223a2267cbeba5726dc912b48d6e017e580
Reviewed-by: Allan Sandfeld Jensen <allan.jensen@qt.io>
Diffstat (limited to 'src')
-rw-r--r-- | src/corelib/tools/qmargins.cpp | 12 | ||||
-rw-r--r-- | src/corelib/tools/qmargins.h | 59 | ||||
-rw-r--r-- | src/corelib/tools/qrect.cpp | 12 | ||||
-rw-r--r-- | src/corelib/tools/qrect.h | 6 |
4 files changed, 34 insertions, 55 deletions
diff --git a/src/corelib/tools/qmargins.cpp b/src/corelib/tools/qmargins.cpp index a238032f65..62ccfff792 100644 --- a/src/corelib/tools/qmargins.cpp +++ b/src/corelib/tools/qmargins.cpp @@ -143,15 +143,13 @@ QT_BEGIN_NAMESPACE */ /*! - \fn bool operator==(const QMargins &m1, const QMargins &m2) - \relates QMargins + \fn bool QMargins::operator==(const QMargins &m1, const QMargins &m2) Returns \c true if \a m1 and \a m2 are equal; otherwise returns \c false. */ /*! - \fn bool operator!=(const QMargins &m1, const QMargins &m2) - \relates QMargins + \fn bool QMargins::operator!=(const QMargins &m1, const QMargins &m2) Returns \c true if \a m1 and \a m2 are different; otherwise returns \c false. */ @@ -566,15 +564,13 @@ QDebug operator<<(QDebug dbg, const QMargins &m) */ /*! - \fn bool operator==(const QMarginsF &lhs, const QMarginsF &rhs) - \relates QMarginsF + \fn bool QMarginsF::operator==(const QMarginsF &lhs, const QMarginsF &rhs) Returns \c true if \a lhs and \a rhs are equal; otherwise returns \c false. */ /*! - \fn bool operator!=(const QMarginsF &lhs, const QMarginsF &rhs) - \relates QMarginsF + \fn bool QMarginsF::operator!=(const QMarginsF &lhs, const QMarginsF &rhs) Returns \c true if \a lhs and \a rhs are different; otherwise returns \c false. */ diff --git a/src/corelib/tools/qmargins.h b/src/corelib/tools/qmargins.h index de7a96e458..f8fba1e9d6 100644 --- a/src/corelib/tools/qmargins.h +++ b/src/corelib/tools/qmargins.h @@ -81,8 +81,19 @@ private: int m_right; int m_bottom; - friend constexpr inline bool operator==(const QMargins &, const QMargins &) noexcept; - friend constexpr inline bool operator!=(const QMargins &, const QMargins &) noexcept; + friend constexpr inline bool operator==(const QMargins &m1, const QMargins &m2) noexcept + { + return + m1.m_left == m2.m_left && + m1.m_top == m2.m_top && + m1.m_right == m2.m_right && + m1.m_bottom == m2.m_bottom; + } + + friend constexpr inline bool operator!=(const QMargins &m1, const QMargins &m2) noexcept + { + return !(m1 == m2); + } }; Q_DECLARE_TYPEINFO(QMargins, Q_MOVABLE_TYPE); @@ -132,24 +143,6 @@ constexpr inline void QMargins::setRight(int aright) noexcept constexpr inline void QMargins::setBottom(int abottom) noexcept { m_bottom = abottom; } -constexpr inline bool operator==(const QMargins &m1, const QMargins &m2) noexcept -{ - return - m1.m_left == m2.m_left && - m1.m_top == m2.m_top && - m1.m_right == m2.m_right && - m1.m_bottom == m2.m_bottom; -} - -constexpr inline bool operator!=(const QMargins &m1, const QMargins &m2) noexcept -{ - return - m1.m_left != m2.m_left || - m1.m_top != m2.m_top || - m1.m_right != m2.m_right || - m1.m_bottom != m2.m_bottom; -} - constexpr inline QMargins operator+(const QMargins &m1, const QMargins &m2) noexcept { return QMargins(m1.left() + m2.left(), m1.top() + m2.top(), @@ -321,6 +314,19 @@ private: qreal m_top; qreal m_right; qreal m_bottom; + + friend constexpr inline bool operator==(const QMarginsF &lhs, const QMarginsF &rhs) noexcept + { + return qFuzzyCompare(lhs.left(), rhs.left()) + && qFuzzyCompare(lhs.top(), rhs.top()) + && qFuzzyCompare(lhs.right(), rhs.right()) + && qFuzzyCompare(lhs.bottom(), rhs.bottom()); + } + + friend constexpr inline bool operator!=(const QMarginsF &lhs, const QMarginsF &rhs) noexcept + { + return !(lhs == rhs); + } }; Q_DECLARE_TYPEINFO(QMarginsF, Q_MOVABLE_TYPE); @@ -375,19 +381,6 @@ constexpr inline void QMarginsF::setRight(qreal aright) noexcept constexpr inline void QMarginsF::setBottom(qreal abottom) noexcept { m_bottom = abottom; } -constexpr inline bool operator==(const QMarginsF &lhs, const QMarginsF &rhs) noexcept -{ - return qFuzzyCompare(lhs.left(), rhs.left()) - && qFuzzyCompare(lhs.top(), rhs.top()) - && qFuzzyCompare(lhs.right(), rhs.right()) - && qFuzzyCompare(lhs.bottom(), rhs.bottom()); -} - -constexpr inline bool operator!=(const QMarginsF &lhs, const QMarginsF &rhs) noexcept -{ - return !operator==(lhs, rhs); -} - constexpr inline QMarginsF operator+(const QMarginsF &lhs, const QMarginsF &rhs) noexcept { return QMarginsF(lhs.left() + rhs.left(), lhs.top() + rhs.top(), diff --git a/src/corelib/tools/qrect.cpp b/src/corelib/tools/qrect.cpp index 326b0c0cd8..d38e532efe 100644 --- a/src/corelib/tools/qrect.cpp +++ b/src/corelib/tools/qrect.cpp @@ -1141,8 +1141,7 @@ bool QRect::intersects(const QRect &r) const noexcept } /*! - \fn bool operator==(const QRect &r1, const QRect &r2) - \relates QRect + \fn bool QRect::operator==(const QRect &r1, const QRect &r2) Returns \c true if the rectangles \a r1 and \a r2 are equal, otherwise returns \c false. @@ -1150,8 +1149,7 @@ bool QRect::intersects(const QRect &r) const noexcept /*! - \fn bool operator!=(const QRect &r1, const QRect &r2) - \relates QRect + \fn bool QRect::operator!=(const QRect &r1, const QRect &r2) Returns \c true if the rectangles \a r1 and \a r2 are different, otherwise returns \c false. @@ -2368,8 +2366,7 @@ QRect QRectF::toAlignedRect() const noexcept */ /*! - \fn bool operator==(const QRectF &r1, const QRectF &r2) - \relates QRectF + \fn bool QRectF::operator==(const QRectF &r1, const QRectF &r2) Returns \c true if the rectangles \a r1 and \a r2 are \b approximately equal, otherwise returns \c false. @@ -2382,8 +2379,7 @@ QRect QRectF::toAlignedRect() const noexcept /*! - \fn bool operator!=(const QRectF &r1, const QRectF &r2) - \relates QRectF + \fn bool QRectF::operator!=(const QRectF &r1, const QRectF &r2) Returns \c true if the rectangles \a r1 and \a r2 are sufficiently different, otherwise returns \c false. diff --git a/src/corelib/tools/qrect.h b/src/corelib/tools/qrect.h index 23ace7d42d..14cab1137e 100644 --- a/src/corelib/tools/qrect.h +++ b/src/corelib/tools/qrect.h @@ -166,9 +166,6 @@ private: }; Q_DECLARE_TYPEINFO(QRect, Q_MOVABLE_TYPE); -constexpr inline bool operator==(const QRect &, const QRect &) noexcept; -constexpr inline bool operator!=(const QRect &, const QRect &) noexcept; - /***************************************************************************** QRect stream functions @@ -630,9 +627,6 @@ private: }; Q_DECLARE_TYPEINFO(QRectF, Q_MOVABLE_TYPE); -constexpr inline bool operator==(const QRectF &, const QRectF &) noexcept; -constexpr inline bool operator!=(const QRectF &, const QRectF &) noexcept; - /***************************************************************************** QRectF stream functions |