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, 1 insertions, 5 deletions

diff --git a/src/quick/handlers/qquickdraghandler.cpp b/src/quick/handlers/qquickdraghandler.cpp
index 7c175b472b..d9f0e477d1 100644
--- a/src/quick/handlers/qquickdraghandler.cpp
+++ b/src/quick/handlers/qquickdraghandler.cpp
@@ -88,15 +88,11 @@ Q_LOGGING_CATEGORY(lcDragHandler, "qt.quick.handler.drag")
     \sa Drag, MouseArea
 */
 
-QQuickDragHandler::QQuickDragHandler(QObject *parent)
+QQuickDragHandler::QQuickDragHandler(QQuickItem *parent)
     : QQuickMultiPointHandler(parent, 1, 1)
 {
 }
 
-QQuickDragHandler::~QQuickDragHandler()
-{
-}
-
 bool QQuickDragHandler::targetContainsCentroid()
 {
     Q_ASSERT(parentItem() && target());