diff options
author | Maximilian Goldstein <max.goldstein@qt.io> | 2021-09-23 16:09:18 +0200 |
---|---|---|
committer | Maximilian Goldstein <max.goldstein@qt.io> | 2021-09-28 11:14:58 +0200 |
commit | 3306225410abea1ec2484dc7f839915958f89c3b (patch) | |
tree | 7a699d15abe04581ce06a1c7a4482afc53640a75 /src/qmlcompiler/qqmljsimportvisitor.cpp | |
parent | 828cbe57a1690c843f4e3de258d465160e599f95 (diff) |
qmllint: Add ability to warn about signal handlers
Another warning ability we need in order to replace controls' tst_sanity with something more sensible.
Probably not useful outside of that, so it is disabled by default.
Fixes: QTBUG-96570
Change-Id: I6ba61897d94a237f029cf9e24dc63d79efcd88ab
Reviewed-by: Ulf Hermann <ulf.hermann@qt.io>
Diffstat (limited to 'src/qmlcompiler/qqmljsimportvisitor.cpp')
-rw-r--r-- | src/qmlcompiler/qqmljsimportvisitor.cpp | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/src/qmlcompiler/qqmljsimportvisitor.cpp b/src/qmlcompiler/qqmljsimportvisitor.cpp index 37943433a2..5664552ccc 100644 --- a/src/qmlcompiler/qqmljsimportvisitor.cpp +++ b/src/qmlcompiler/qqmljsimportvisitor.cpp @@ -1312,6 +1312,10 @@ bool QQmlJSImportVisitor::visit(UiScriptBinding *scriptBinding) signalParameters << formal->element->bindingIdentifier.toString(); } } + + m_logger->logWarning(u"Declared signal handler \"%1\""_qs.arg(name), Log_ControlsSanity, + scriptBinding->firstSourceLocation()); + m_signals[m_currentScope].append({ m_savedBindingOuterScope, group->firstSourceLocation(), qMakePair(name.toString(), signalParameters) }); |