diff options
author | Arno Rehn <a.rehn@menlosystems.com> | 2019-01-15 18:08:25 +0100 |
---|---|---|
committer | Arno Rehn <a.rehn@menlosystems.com> | 2019-01-23 11:43:29 +0000 |
commit | c80cbce7ac3b38a45a38b57795c7eb19e5177a17 (patch) | |
tree | a42ad1d1cc5ae88932fdd2bbf682934b2f39fb1b | |
parent | 45bc3ab0bbc292be26b2bf0596487d9c7f2059d3 (diff) | |
download | qtwebchannel-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.cpp | 5 |
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 ¬ifySignal = prop.notifySignal().name(); static const QByteArray changedSuffix = QByteArrayLiteral("Changed"); |