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/qquickpointhandler.cpp b/src/quick/handlers/qquickpointhandler.cpp
index 65a7fd7b79..da19584b7a 100644
--- a/src/quick/handlers/qquickpointhandler.cpp
+++ b/src/quick/handlers/qquickpointhandler.cpp
@@ -114,16 +114,12 @@ QT_BEGIN_NAMESPACE
     \sa MultiPointTouchArea
 */
 
-QQuickPointHandler::QQuickPointHandler(QObject *parent)
+QQuickPointHandler::QQuickPointHandler(QQuickItem *parent)
     : QQuickSinglePointHandler(parent)
 {
     setIgnoreAdditionalPoints();
 }
 
-QQuickPointHandler::~QQuickPointHandler()
-{
-}
-
 bool QQuickPointHandler::wantsEventPoint(QQuickEventPoint *pt)
 {
     // On press, we want it unless a sibling of the same type also does.