From: eap <eap@opencascade.com>
Date: Fri, 10 Jan 2014 07:54:43 +0000 (+0000)
Subject: 22455: Error "No mesh elements assigned to a face" is raised instead of a simple... 
X-Git-Tag: V7_4_0a1~7
X-Git-Url: http://git.salome-platform.org/gitweb/?a=commitdiff_plain;h=8291a5c9f274fccb99074f12c341a29363ba31ef;p=plugins%2Fblsurfplugin.git

22455: Error "No mesh elements assigned to a face" is raised instead of a simple warning
---

diff --git a/src/BLSURFPlugin/BLSURFPlugin_BLSURF.cxx b/src/BLSURFPlugin/BLSURFPlugin_BLSURF.cxx
index a146e6a..2ba902c 100644
--- a/src/BLSURFPlugin/BLSURFPlugin_BLSURF.cxx
+++ b/src/BLSURFPlugin/BLSURFPlugin_BLSURF.cxx
@@ -2747,10 +2747,6 @@ bool BLSURFPlugin_BLSURF::compute(SMESH_Mesh&         aMesh,
     if ( _comment.empty() )
       _comment = "Exception in cadsurf_compute_mesh()";
   }
-  if ( status != STATUS_OK) {
-    // There was an error while meshing
-    error(_comment);
-  }
 
   std::cout << std::endl;
   std::cout << "End of Surface Mesh generation" << std::endl;
@@ -3073,16 +3069,27 @@ bool BLSURFPlugin_BLSURF::compute(SMESH_Mesh&         aMesh,
 
   // Set error to FACE's w/o elements
   SMESH_ComputeErrorName err = COMPERR_ALGO_FAILED;
-  if ( _comment.empty() )
+  if ( _comment.empty() && status == STATUS_OK )
   {
     err      = COMPERR_WARNING;
     _comment = "No mesh elements assigned to a face";
   }
+  bool badFaceFound = false;
   for ( int i = 1; i <= fmap.Extent(); ++i )
   {
     SMESH_subMesh* sm = aMesh.GetSubMesh( fmap(i) );
     if ( !sm->GetSubMeshDS() || sm->GetSubMeshDS()->NbElements() == 0 )
+    {
       sm->GetComputeError().reset( new SMESH_ComputeError( err, _comment, this ));
+      badFaceFound = true;
+    }
+  }
+  if ( err == COMPERR_WARNING )
+  {
+    _comment.clear();
+  }
+  if ( status != STATUS_OK && !badFaceFound ) {
+    error(_comment);
   }
 
   // Issue 0019864. On DebianSarge, FE signals do not obey to OSD::SetSignal(false)
@@ -3376,7 +3383,10 @@ status_t message_cb(message_t *msg, void *user_data)
   string err( desc );
   message_cb_user_data * mcud = (message_cb_user_data*)user_data;
   // Get all the error message and some warning messages related to license and periodicity
-  if ( errnumber < 0 || err.find("license") != string::npos || err.find("periodicity") != string::npos ) {
+  if ( errnumber < 0 ||
+       err.find("license"    ) != string::npos ||
+       err.find("periodicity") != string::npos )
+  {
     // remove ^A from the tail
     int len = strlen( desc );
     while (len > 0 && desc[len-1] != '\n')