qtcassert: move actual printing to separate function and enforce style

This also allows simple setting of breakpoints on failed asserts.

Change-Id: I6dd84cbfaf659d57e39f3447386cebc0221b2b84
Reviewed-by: Daniel Teske <daniel.teske@nokia.com>

1 files changed, 2 insertions, 3 deletions

diff --git a/src/plugins/valgrind/callgrind/callgrindcontroller.cpp b/src/plugins/valgrind/callgrind/callgrindcontroller.cpp
index f3281fed4b..ea765081bf 100644
--- a/src/plugins/valgrind/callgrind/callgrindcontroller.cpp
+++ b/src/plugins/valgrind/callgrind/callgrindcontroller.cpp
@@ -94,14 +94,13 @@ QString toOptionString(CallgrindController::Option option)
     }
 }
 
-
 void CallgrindController::run(Option option)
 {
     if (m_process) {
         emit statusMessage(tr("Previous command has not yet finished."));
         return;
     }
-    QTC_ASSERT(m_valgrindProc, return)
+    QTC_ASSERT(m_valgrindProc, return);
 
     if (RemoteValgrindProcess *remote = qobject_cast<RemoteValgrindProcess *>(m_valgrindProc))
         m_process = new RemoteValgrindProcess(remote->connection(), this);
@@ -150,7 +149,7 @@ void CallgrindController::run(Option option)
 
 void CallgrindController::processError(QProcess::ProcessError)
 {
-    QTC_ASSERT(m_process, return)
+    QTC_ASSERT(m_process, return);
     const QString error = m_process->errorString();
     emit statusMessage(QString("An error occurred while trying to run %1: %2").arg(CALLGRIND_CONTROL_BINARY).arg(error));