Fix some clang-tidy warnings

- Fix 'if (foo) delete foo' to 'delete foo'
- Use ' = default' for trivial constructors/destructors
- Use range based for where possible
- Fix invocations of static methods
- Replace some reinterpret_cast by static_cast

Change-Id: I998572f370f3e8a1fc39e6c763e941d5a2f598b0
Reviewed-by: Oliver Wolff <oliver.wolff@qt.io>

1 files changed, 2 insertions, 5 deletions

diff --git a/src/winextras/qwintaskbarbutton.cpp b/src/winextras/qwintaskbarbutton.cpp
index 2516865..a1475d3 100644
--- a/src/winextras/qwintaskbarbutton.cpp
+++ b/src/winextras/qwintaskbarbutton.cpp
@@ -152,8 +152,7 @@ void QWinTaskbarButtonPrivate::updateOverlayIcon()
 
     if (hicon)
         DestroyIcon(hicon);
-    if (descrPtr)
-        delete[] descrPtr;
+    delete[] descrPtr;
 }
 
 void QWinTaskbarButtonPrivate::_q_updateProgress()
@@ -189,9 +188,7 @@ QWinTaskbarButton::QWinTaskbarButton(QObject *parent) :
 /*!
     Destroys the QWinTaskbarButton.
  */
-QWinTaskbarButton::~QWinTaskbarButton()
-{
-}
+QWinTaskbarButton::~QWinTaskbarButton() = default;
 
 /*!
     \property QWinTaskbarButton::window