From 291571284292e81c23c2dbdfec6c47907de1daea Mon Sep 17 00:00:00 2001
From: blobfish <blobfishATgmx.com>
Date: Thu, 20 Feb 2014 07:51:56 -0500
Subject: [PATCH] Part: CheckGeometry: Mantis #0001426

    Empty compound throwing exception for bounding box.
---
 src/Mod/Part/Gui/TaskCheckGeometry.cpp | 73 ++++++++++++++------------
 1 file changed, 39 insertions(+), 34 deletions(-)

diff --git a/src/Mod/Part/Gui/TaskCheckGeometry.cpp b/src/Mod/Part/Gui/TaskCheckGeometry.cpp
index 5105b138cdac..d16e64bcd7f3 100644
--- a/src/Mod/Part/Gui/TaskCheckGeometry.cpp
+++ b/src/Mod/Part/Gui/TaskCheckGeometry.cpp
@@ -737,40 +737,45 @@ void PartGui::goSetupResultTypedSelection(ResultEntry* entry, const TopoDS_Shape
 
 void PartGui::goSetupResultBoundingBox(ResultEntry *entry)
 {
-    entry->boxSep = new SoSeparator();
-    entry->viewProvider->getRoot()->addChild(entry->boxSep);
-    entry->boxSwitch = new SoSwitch();
-    entry->boxSep->addChild(entry->boxSwitch);
-    SoGroup *group = new SoGroup();
-    entry->boxSwitch->addChild(group);
-    entry->boxSwitch->whichChild.setValue(SO_SWITCH_NONE);
-    SoDrawStyle *dStyle = new SoDrawStyle();
-    dStyle->style.setValue(SoDrawStyle::LINES);
-    dStyle->linePattern.setValue(0xc0c0);
-    group->addChild(dStyle);
-    SoMaterial *material = new SoMaterial();
-    material->diffuseColor.setValue(255.0, 255.0, 255.0);
-    material->ambientColor.setValue(255.0, 255.0, 255.0);
-    group->addChild(material);
-
-    SoResetTransform *reset = new SoResetTransform();
-    group->addChild(reset);
-    
-    Bnd_Box boundingBox;
-    BRepBndLib::Add(entry->shape, boundingBox);
-    Standard_Real xmin, ymin, zmin, xmax, ymax, zmax;
-    boundingBox.Get(xmin, ymin, zmin, xmax, ymax, zmax);
-    SbVec3f boundCenter((xmax - xmin)/2 + xmin, (ymax - ymin)/2 + ymin, (zmax - zmin)/2 + zmin);
-
-    SoTransform *position = new SoTransform();
-    position->translation.setValue(boundCenter);
-    group->addChild(position);
-
-    SoCube *cube = new SoCube();
-    cube->width.setValue(xmax - xmin);
-    cube->height.setValue(ymax - ymin);
-    cube->depth.setValue(zmax - zmin);
-    group->addChild(cube);
+    //empty compound throws bounding box error. Mantis #0001426
+    try
+    {
+      Bnd_Box boundingBox;
+      BRepBndLib::Add(entry->shape, boundingBox);
+      Standard_Real xmin, ymin, zmin, xmax, ymax, zmax;
+      boundingBox.Get(xmin, ymin, zmin, xmax, ymax, zmax);
+      SbVec3f boundCenter((xmax - xmin)/2 + xmin, (ymax - ymin)/2 + ymin, (zmax - zmin)/2 + zmin);
+  
+      entry->boxSep = new SoSeparator();
+      entry->viewProvider->getRoot()->addChild(entry->boxSep);
+      entry->boxSwitch = new SoSwitch();
+      entry->boxSep->addChild(entry->boxSwitch);
+      SoGroup *group = new SoGroup();
+      entry->boxSwitch->addChild(group);
+      entry->boxSwitch->whichChild.setValue(SO_SWITCH_NONE);
+      SoDrawStyle *dStyle = new SoDrawStyle();
+      dStyle->style.setValue(SoDrawStyle::LINES);
+      dStyle->linePattern.setValue(0xc0c0);
+      group->addChild(dStyle);
+      SoMaterial *material = new SoMaterial();
+      material->diffuseColor.setValue(255.0, 255.0, 255.0);
+      material->ambientColor.setValue(255.0, 255.0, 255.0);
+      group->addChild(material);
+
+      SoResetTransform *reset = new SoResetTransform();
+      group->addChild(reset);
+      
+      SoTransform *position = new SoTransform();
+      position->translation.setValue(boundCenter);
+      group->addChild(position);
+
+      SoCube *cube = new SoCube();
+      cube->width.setValue(xmax - xmin);
+      cube->height.setValue(ymax - ymin);
+      cube->depth.setValue(zmax - zmin);
+      group->addChild(cube);
+    }
+    catch (const Standard_Failure &){}
 }
 
 void PartGui::goSetupResultShellNotClosed(ResultEntry *entry)