aboutsummaryrefslogtreecommitdiffstats
path: root/src
diff options
context:
space:
mode:
authorSergio Martins <smartins@kde.org>2019-09-29 23:43:31 +0100
committerSergio Martins <smartins@kde.org>2019-09-29 23:43:31 +0100
commitfe237dd5b035ae6509da31268f9e255127a6558a (patch)
tree0e7ba59e45573bf485ff7e28e7778500c9e44a10 /src
parentfa01953777c8f62862d4f57d465771f3776dad06 (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.cpp1
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;
}
}