Fix PointerHandler constructors and destructors

- Constructors should take QQuickItem* not QObject* to be symmetric
  with the parentItem() accessor (and other code) which assumes its type
- Use header initialization everywhere possible
- Reorder variables to minimize padding (somewhat)
- Remove empty destructor bodies (the compiler can write them)
- Remove override and virtual from destructors in accordance with
  https://github.com/isocpp/CppCoreGuidelines/blob/master/CppCoreGuidelines.md#Rh-override

Change-Id: I682a53a803d65e29136bfaec3a5b534e975ecf30
Reviewed-by: Jan Arve Sæther <jan-arve.saether@qt.io>

1 files changed, 2 insertions, 6 deletions

diff --git a/src/quick/handlers/qquickpointerdevicehandler.cpp b/src/quick/handlers/qquickpointerdevicehandler.cpp
index dfea3ec02d..24b3dc4733 100644
--- a/src/quick/handlers/qquickpointerdevicehandler.cpp
+++ b/src/quick/handlers/qquickpointerdevicehandler.cpp
@@ -57,20 +57,16 @@ QT_BEGIN_NAMESPACE
     An intermediate class (not registered as a QML type) for handlers which
     allow filtering based on device type, pointer type, or keyboard modifiers.
 */
-QQuickPointerDeviceHandler::QQuickPointerDeviceHandler(QObject *parent)
+QQuickPointerDeviceHandler::QQuickPointerDeviceHandler(QQuickItem *parent)
     : QQuickPointerHandler(*(new QQuickPointerDeviceHandlerPrivate), parent)
 {
 }
 
-QQuickPointerDeviceHandler::QQuickPointerDeviceHandler(QQuickPointerDeviceHandlerPrivate &dd, QObject *parent)
+QQuickPointerDeviceHandler::QQuickPointerDeviceHandler(QQuickPointerDeviceHandlerPrivate &dd, QQuickItem *parent)
     : QQuickPointerHandler(dd, parent)
 {
 }
 
-QQuickPointerDeviceHandler::~QQuickPointerDeviceHandler()
-{
-}
-
 QQuickPointerDevice::DeviceTypes QQuickPointerDeviceHandler::acceptedDevices() const
 {
     Q_D(const QQuickPointerDeviceHandler);