Merge pull request #2767 from ThorstenHarter/master

Fix regression in pcl::SACSegmentation line fitting
PointCloudLibrary · Jan 10, 2019 · f3cc278 · f3cc278
2 parents a023a4f + 36c2bd6
commit f3cc278
Showing 1 changed file with 5 additions and 2 deletions.
diff --git a/sample_consensus/include/pcl/sample_consensus/impl/sac_model_line.hpp b/sample_consensus/include/pcl/sample_consensus/impl/sac_model_line.hpp
@@ -49,13 +49,16 @@
 template <typename PointT> bool
 pcl::SampleConsensusModelLine<PointT>::isSampleGood (const std::vector<int> &samples) const
 {
+  // Make sure that the two sample points are not identical
   if (
       (input_->points[samples[0]].x != input_->points[samples[1]].x)
-    &&
+    ||
       (input_->points[samples[0]].y != input_->points[samples[1]].y)
-    &&
+    ||
       (input_->points[samples[0]].z != input_->points[samples[1]].z))
+  {
     return (true);
+  }
 
   return (false);
 }