Merge pull request #387 from lmontaut/devel

[GJKSolver] Fix missing switch case in result status of GJK
coal-library · Mar 24, 2023 · a8b8c2f · a8b8c2f
2 parents 98303de + 872ff4c
commit a8b8c2f
Showing 1 changed file with 2 additions and 1 deletion.
diff --git a/include/hpp/fcl/narrowphase/narrowphase.h b/include/hpp/fcl/narrowphase/narrowphase.h
@@ -235,6 +235,7 @@ struct HPP_FCL_DLLAPI GJKSolver {
         }
         break;
       case details::GJK::Valid:
+      case details::GJK::EarlyStopped:
       case details::GJK::Failed:
         col = false;
 
@@ -250,7 +251,7 @@ struct HPP_FCL_DLLAPI GJKSolver {
         break;
       default:
         assert(false && "should not reach type part.");
-        return true;
+        throw std::logic_error("GJKSolver: should not reach this part.");
     }
     return col;
   }