diff options
author | Sergio Martins <smartins@kde.org> | 2017-09-21 00:04:03 +0100 |
---|---|---|
committer | Sergio Martins <smartins@kde.org> | 2017-09-21 00:04:03 +0100 |
commit | 200b2ecb4acbd8701905d0b571c6bc3ba9687da7 (patch) | |
tree | 327e137037434a7132fac35cf64cdcdea063c942 /tests/virtual-signal | |
parent | 55e28db5a89dda34cf60227efe69a396e406b9d5 (diff) |
Introducing virtual-signal
Warns when a signal is virtual.
moc already warns about this, but it's not a fatal warning.
With clazy you can add -Werror.
Diffstat (limited to 'tests/virtual-signal')
-rw-r--r-- | tests/virtual-signal/config.json | 7 | ||||
-rw-r--r-- | tests/virtual-signal/main.cpp | 14 | ||||
-rw-r--r-- | tests/virtual-signal/main.cpp.expected | 1 |
3 files changed, 22 insertions, 0 deletions
diff --git a/tests/virtual-signal/config.json b/tests/virtual-signal/config.json new file mode 100644 index 00000000..e7e6e0cb --- /dev/null +++ b/tests/virtual-signal/config.json @@ -0,0 +1,7 @@ +{ + "tests" : [ + { + "filename" : "main.cpp" + } + ] +} diff --git a/tests/virtual-signal/main.cpp b/tests/virtual-signal/main.cpp new file mode 100644 index 00000000..1bdd0f9a --- /dev/null +++ b/tests/virtual-signal/main.cpp @@ -0,0 +1,14 @@ +#include <QtCore/QObject> + +class MyObj : public QObject +{ + Q_OBJECT +public: + void foo(); +signals: + void mySig(); // OK + virtual void myVirtualSig(); // Warn +public Q_SLOTS: + void mySlot(); +}; + diff --git a/tests/virtual-signal/main.cpp.expected b/tests/virtual-signal/main.cpp.expected new file mode 100644 index 00000000..9110dc37 --- /dev/null +++ b/tests/virtual-signal/main.cpp.expected @@ -0,0 +1 @@ +virtual-signal/main.cpp:10:5: warning: signal is virtual [-Wclazy-virtual-signal] |