diff options
author | Sergio Martins <smartins@kde.org> | 2019-09-29 23:43:31 +0100 |
---|---|---|
committer | Sergio Martins <smartins@kde.org> | 2019-09-29 23:43:31 +0100 |
commit | fe237dd5b035ae6509da31268f9e255127a6558a (patch) | |
tree | 0e7ba59e45573bf485ff7e28e7778500c9e44a10 /src | |
parent | fa01953777c8f62862d4f57d465771f3776dad06 (diff) |
qproperty-type-mismatch: Don't warn if notify signal has too many params
having more than 1 param is commonplace and not a bug.
Although elegant to only have 1, if we warn we'll produce lots of noise
and people will probably disable this check instead, failing to find more
important bugs
Diffstat (limited to 'src')
-rw-r--r-- | src/checks/manuallevel/qproperty-type-mismatch.cpp | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/src/checks/manuallevel/qproperty-type-mismatch.cpp b/src/checks/manuallevel/qproperty-type-mismatch.cpp index ca246ea0..1e45804a 100644 --- a/src/checks/manuallevel/qproperty-type-mismatch.cpp +++ b/src/checks/manuallevel/qproperty-type-mismatch.cpp @@ -178,7 +178,6 @@ void QPropertyTypeMismatch::checkMethodAgainstProperty (const Property& prop, co } default: { - emitWarning(&method, error_begin() + "signal '" + methodName + "' with too many parameters"); break; } } |