summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorArno Rehn <a.rehn@menlosystems.com>2019-01-15 18:08:25 +0100
committerArno Rehn <a.rehn@menlosystems.com>2019-01-23 11:43:29 +0000
commitc80cbce7ac3b38a45a38b57795c7eb19e5177a17 (patch)
treea42ad1d1cc5ae88932fdd2bbf682934b2f39fb1b
parent45bc3ab0bbc292be26b2bf0596487d9c7f2059d3 (diff)
downloadqtwebchannel-c80cbce7ac3b38a45a38b57795c7eb19e5177a17.tar.gz
Remove warning when the notify signal has more than one argument
The warning was not meaningful, since nothing depends on the notify signal having only zero or one argument(s). Removing this decreases warning noise in some applications. Change-Id: Ieb716ad9ac51296a1e17899018d4e6190043b8d5 Reviewed-by: Frederik Gladhorn <frederik.gladhorn@qt.io>
-rw-r--r--src/webchannel/qmetaobjectpublisher.cpp5
1 files changed, 0 insertions, 5 deletions
diff --git a/src/webchannel/qmetaobjectpublisher.cpp b/src/webchannel/qmetaobjectpublisher.cpp
index 8e83237..9e08453 100644
--- a/src/webchannel/qmetaobjectpublisher.cpp
+++ b/src/webchannel/qmetaobjectpublisher.cpp
@@ -149,11 +149,6 @@ QJsonObject QMetaObjectPublisher::classInfoForObject(const QObject *object, QWeb
QJsonArray signalInfo;
if (prop.hasNotifySignal()) {
notifySignals << prop.notifySignalIndex();
- const int numParams = prop.notifySignal().parameterCount();
- if (numParams > 1) {
- qWarning("Notify signal for property '%s' has %d parameters, expected zero or one.",
- prop.name(), numParams);
- }
// optimize: compress the common propertyChanged notification names, just send a 1
const QByteArray &notifySignal = prop.notifySignal().name();
static const QByteArray changedSuffix = QByteArrayLiteral("Changed");