Hide comparison operators for QtNetwork value types from non-ADL

Make them hidden friends, add a private isEqual helper where needed.
Adjust and add documentation.

Fixes: QTBUG-87976
Change-Id: If7c19eeab5be7452364eb76193981100f5516d6b
Reviewed-by: Allan Sandfeld Jensen <allan.jensen@qt.io>

1 files changed, 17 insertions, 4 deletions

diff --git a/src/network/access/qhstspolicy.cpp b/src/network/access/qhstspolicy.cpp
index 46c9c22510..3d15cba407 100644
--- a/src/network/access/qhstspolicy.cpp
+++ b/src/network/access/qhstspolicy.cpp
@@ -86,12 +86,25 @@ public:
 };
 
 /*!
-    Returns \c true if the two policies have the same host and expiration date
-    while agreeing on whether to include or exclude subdomains.
+    \fn bool QHstsPolicy::operator==(const QHstsPolicy &lhs, const QHstsPolicy &rhs)
+
+    Returns \c true if the two policies \a lhs and \a rhs have the same host and
+    expiration date while agreeing on whether to include or exclude subdomains.
+*/
+
+/*!
+    \fn bool QHstsPolicy::operator!=(const QHstsPolicy &lhs, const QHstsPolicy &rhs)
+
+    Returns \c true if the two policies \a lhs and \a rhs do not have the same host
+    or expiration date, or do not agree on whether to include or exclude subdomains.
+*/
+
+/*!
+    \internal
 */
-bool operator==(const QHstsPolicy &lhs, const QHstsPolicy &rhs)
+bool QHstsPolicy::isEqual(const QHstsPolicy &other) const
 {
-    return *lhs.d == *rhs.d;
+    return *d == *other.d;
 }
 
 /*!