diff options
author | Thiago Macieira <thiago.macieira@intel.com> | 2014-12-17 18:59:17 -0800 |
---|---|---|
committer | Thiago Macieira <thiago.macieira@intel.com> | 2014-12-20 07:38:21 +0100 |
commit | 3092bd56c394f22e027be59368789546afb5ca98 (patch) | |
tree | 2fa6824f25630efe3004a6226641e2307ce0f1d2 /src/corelib/global | |
parent | 2e4b1a8334d4de585e90637c070ba9832a0608f1 (diff) |
Fix Clang warning about inconsistent use of C++11 override
We cannot use override in the Q_OBJECT macro, so tell Clang not to warn
about our lack of use of the keyword in the three virtuals that Q_OBJECT
overrides when the user uses Q_DECL_OVERRIDE in their own functions.
We can't use Q_DECL_OVERRIDE for two reasons:
1) because that would trigger the warning in any and all virtual
overrides the user may have in their own class (and most of Qt)
2) because when Q_OBJECT is used in QObject, it's not overriding
anything -- it's the base declaration
For #2, we could add a different macro exclusively for QObject that moc
would expand, but it's unnecessary work.
Change-Id: Ic9cb7896f93f06f4e86b6a42dc08f96d257f7550
Reviewed-by: Jędrzej Nowacki <jedrzej.nowacki@theqtcompany.com>
Reviewed-by: Olivier Goffart <ogoffart@woboq.com>
Diffstat (limited to 'src/corelib/global')
0 files changed, 0 insertions, 0 deletions