Avoid warning on empty expressions

For some reason, Q_UNUSED includes already a semicolon, adding one
on the user side creates an additional empty statement.

Change-Id: I9c5e8fac381345a60792cb75e2938fd53958d3b0
Reviewed-by: Christian Stenger <christian.stenger@qt.io>

src/plugins/valgrind/callgrind/callgrindcontroller.cpp

@@ -187,7 +187,7 @@ void CallgrindController::controllerProcessFinished(int rc, QProcess::ExitStatus
 
 void CallgrindController::controllerProcessClosed(bool success)
 {
-    Q_UNUSED(success);
+    Q_UNUSED(success)
     //    QTC_ASSERT(m_remote.m_process, return);
 
 //    m_remote.m_errorString = m_remote.m_process->errorString();

src/plugins/valgrind/memchecktool.cpp

@@ -818,7 +818,7 @@ void MemcheckToolPrivate::heobAction()
     // process environment
     QByteArray env;
     void *envPtr = nullptr;
-    Q_UNUSED(envPtr);
+    Q_UNUSED(envPtr)
     if (!envStrings.isEmpty()) {
         uint pos = 0;
         for (const QString &par : envStrings) {

src/plugins/valgrind/suppressiondialog.cpp

@@ -227,7 +227,7 @@ void SuppressionDialog::accept()
     foreach (const QModelIndex &index, indices) {
         bool removed = model->removeRow(index.row());
         QTC_ASSERT(removed, qt_noop());
-        Q_UNUSED(removed);
+        Q_UNUSED(removed)
     }
 
     // One suppression might hide multiple rows, care for that.

src/plugins/valgrind/valgrindmemcheckparsertest.h

@@ -123,7 +123,7 @@ public:
     }
     void processErrorReceived(const QString &s)
     {
-        Q_UNUSED(s);
+        Q_UNUSED(s)
         // qDebug() << "error received:" << s; // this can be a lot of text
         m_errorReceived = true;
     }