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, 20 deletions

diff --git a/src/quick/handlers/qquickpinchhandler.cpp b/src/quick/handlers/qquickpinchhandler.cpp
index d6e1a183bb..75a2847aa0 100644
--- a/src/quick/handlers/qquickpinchhandler.cpp
+++ b/src/quick/handlers/qquickpinchhandler.cpp
@@ -85,27 +85,8 @@ Q_LOGGING_CATEGORY(lcPinchHandler, "qt.quick.handler.pinch")
     \sa PinchArea
 */
 
-QQuickPinchHandler::QQuickPinchHandler(QObject *parent)
+QQuickPinchHandler::QQuickPinchHandler(QQuickItem *parent)
     : QQuickMultiPointHandler(parent, 2)
-    , m_activeScale(1)
-    , m_accumulatedScale(1)
-    , m_activeRotation(0)
-    , m_activeTranslation(0,0)
-    , m_minimumScale(-qInf())
-    , m_maximumScale(qInf())
-    , m_minimumRotation(-qInf())
-    , m_maximumRotation(qInf())
-    , m_minimumX(-qInf())
-    , m_maximumX(qInf())
-    , m_minimumY(-qInf())
-    , m_maximumY(qInf())
-    , m_pinchOrigin(PinchCenter)
-    , m_startScale(1)
-    , m_startRotation(0)
-{
-}
-
-QQuickPinchHandler::~QQuickPinchHandler()
 {
 }