From 662184a54fb7e8dd9898622a062daf251b57971e Mon Sep 17 00:00:00 2001 From: Pekka Vuorela Date: Tue, 21 Nov 2017 16:04:06 +0200 Subject: [PATCH] clean up strange if-else structure --- src/transferengine.cpp | 8 +++----- 1 file changed, 3 insertions(+), 5 deletions(-) diff --git a/src/transferengine.cpp b/src/transferengine.cpp index 28cd04f..1a0d8db 100644 --- a/src/transferengine.cpp +++ b/src/transferengine.cpp @@ -363,8 +363,7 @@ void TransferEnginePrivate::sendNotification(TransferEngineData::TransferType ty qWarning() << "TransferEnginePrivate::sendNotification: unknown state"; break; } - } else { - if (status == TransferEngineData::TransferInterrupted) { + } else if (status == TransferEngineData::TransferInterrupted) { category = TRANSFER_ERROR_EVENT_CATEGORY; switch (type) { @@ -391,15 +390,14 @@ void TransferEnginePrivate::sendNotification(TransferEngineData::TransferType ty summary = fileName; previewSummary = summary; previewBody = body; - } else { - if (status == TransferEngineData::TransferCanceled) { + } else if (status == TransferEngineData::TransferCanceled) { // Exit, no banners or events when user has canceled a transfer // Remove any existing notification if (existing) { existing->close(); } - }}} + } if (!category.isEmpty()) { Notification notification;