From 951bf6e3ec7c518dfc6b672cd1f99bcb5a347084 Mon Sep 17 00:00:00 2001 From: Marc Mutz Date: Fri, 22 Mar 2024 08:00:05 +0100 Subject: [PATCH] QSignalSpy: make two private static helpers file-static ... because they can be. As a drive-by, fix clazy-function-args-by-value by taking QMetaMethod by value. Task-number: QTBUG-123544 Change-Id: Icdad68b91850d284c918e6180f3ce841de2af016 Reviewed-by: Edward Welbourne --- src/testlib/qsignalspy.cpp | 4 ++-- src/testlib/qsignalspy.h | 3 --- 2 files changed, 2 insertions(+), 5 deletions(-) diff --git a/src/testlib/qsignalspy.cpp b/src/testlib/qsignalspy.cpp index 52be7ff212..9a5efcd000 100644 --- a/src/testlib/qsignalspy.cpp +++ b/src/testlib/qsignalspy.cpp @@ -161,7 +161,7 @@ bool QSignalSpy::wait(std::chrono::milliseconds timeout) return size() > origCount; } -bool QSignalSpy::isSignalMetaMethodValid(const QMetaMethod &signal) +static bool isSignalMetaMethodValid(QMetaMethod signal) { if (!signal.isValid()) { qWarning("QSignalSpy: Null signal is not valid"); @@ -176,7 +176,7 @@ bool QSignalSpy::isSignalMetaMethodValid(const QMetaMethod &signal) return true; } -bool QSignalSpy::isObjectValid(const QObject *object) +static bool isObjectValid(const QObject *object) { const bool valid = !!object; diff --git a/src/testlib/qsignalspy.h b/src/testlib/qsignalspy.h index cbb3bfbe42..ccde13820a 100644 --- a/src/testlib/qsignalspy.h +++ b/src/testlib/qsignalspy.h @@ -86,9 +86,6 @@ private: Q_TESTLIB_EXPORT bool connectToSignal(const QObject *sender, int sigIndex); - Q_TESTLIB_EXPORT static bool isSignalMetaMethodValid(const QMetaMethod &signal); - Q_TESTLIB_EXPORT static bool isObjectValid(const QObject *object); - Q_TESTLIB_EXPORT static QList makeArgs(const QMetaMethod &member, const QObject *obj); Q_TESTLIB_EXPORT void appendArgs(void **a);