From: akl <alexander.kovalev@opencascade.com>
Date: Tue, 24 Jun 2014 13:20:50 +0000 (+0400)
Subject: Small code refactoring.
X-Git-Tag: V7_5_0a1~50^2
X-Git-Url: http://git.salome-platform.org/gitweb/?a=commitdiff_plain;h=c53193642977279fc6be831334e6d8e0882ef467;p=modules%2Fgeom.git

Small code refactoring.
---

diff --git a/src/GEOMToolsGUI/GEOMToolsGUI_ReduceStudyDlg.cxx b/src/GEOMToolsGUI/GEOMToolsGUI_ReduceStudyDlg.cxx
index 272d63d65..4fa9d4efa 100644
--- a/src/GEOMToolsGUI/GEOMToolsGUI_ReduceStudyDlg.cxx
+++ b/src/GEOMToolsGUI/GEOMToolsGUI_ReduceStudyDlg.cxx
@@ -641,21 +641,16 @@ void GEOMToolsGUI_ReduceStudyDlg::clickOnOk()
     objectsToBeUnpublished.insert( *iter );
   }
   if( myGroupIntermediates->checkedId() == 2 ) { // remove
-	if( myCBSoftRemoval->isChecked() )
-      for( iter = myListParents.begin(); iter != myListParents.end(); ++iter )
-        objectsToBeRemoved.insert( *iter );
-	else {
-      if ( SUIT_MessageBox::question( this,
-                                      tr( "GEOM_WRN_WARNING" ),
-                                      tr( "GEOM_REDUCE_STUDY_WARNING_DELETE" ),
-                                      QMessageBox::Yes | QMessageBox::No,
-                                      QMessageBox::Yes ) == QMessageBox::Yes ) {
-        for( iter = myListParents.begin(); iter != myListParents.end(); ++iter )
-          objectsToBeRemoved.insert( *iter );
-      }
-      else
-        return;
-	}
+    if( !myCBSoftRemoval->isChecked() && 
+	SUIT_MessageBox::question( this,
+				   tr( "GEOM_WRN_WARNING" ),
+				   tr( "GEOM_REDUCE_STUDY_WARNING_DELETE" ),
+				   QMessageBox::Yes | QMessageBox::No,
+				   QMessageBox::Yes ) == QMessageBox::No ) {
+      return;
+    }
+    for( iter = myListParents.begin(); iter != myListParents.end(); ++iter )
+      objectsToBeRemoved.insert( *iter );
   }
 
   // Create lists of sub-objects to be removed or to be unpublished