From 29e604ae7d2d44822da579a2cdb28a7fbe045a9c Mon Sep 17 00:00:00 2001
From: Montel Laurent <montel@kde.org>
Date: Mon, 19 Dec 2011 18:44:26 +0100
Subject: [PATCH] Minor

---
 kmcommands.cpp | 10 ++++------
 1 file changed, 4 insertions(+), 6 deletions(-)

diff --git a/kmcommands.cpp b/kmcommands.cpp
index ffd335fca..c8ab4cbe1 100644
--- a/kmcommands.cpp
+++ b/kmcommands.cpp
@@ -948,8 +948,9 @@ KMCommand::Result KMForwardAttachedCommand::execute()
   if ( !mWin ) {
     mWin = KMail::makeComposer( fwdMsg.first, KMail::Composer::Forward, mIdentity );
   }
-  foreach( KMime::Content* attach, fwdMsg.second )
+  foreach( KMime::Content* attach, fwdMsg.second ) {
     mWin->addAttach( attach );
+  }
   mWin->show();
   return OK;
 }
@@ -1003,11 +1004,8 @@ KMCommand::Result KMRedirectCommand::execute()
 
   const MailTransport::SentBehaviourAttribute *sentAttribute = item.attribute<MailTransport::SentBehaviourAttribute>();
   QString fcc;
-  if ( sentAttribute ) {
-    if ( sentAttribute->sentBehaviour() == MailTransport::SentBehaviourAttribute::MoveToCollection )
-      fcc =  QString::number( sentAttribute->moveToCollection().id() );
-  }
-
+  if ( sentAttribute && ( sentAttribute->sentBehaviour() == MailTransport::SentBehaviourAttribute::MoveToCollection ) )
+    fcc =  QString::number( sentAttribute->moveToCollection().id() );
 
   const KMime::Message::Ptr newMsg = factory.createRedirect( dlg->to(), transportId, fcc );
   if ( !newMsg )