Skip to content

Commit

Permalink
Merge pull request #6031 from PointCloudLibrary/revert-6017-revert_ab…
Browse files Browse the repository at this point in the history
…i_problems_1_14_1

Revert "Fix ABI compatibility problems for 1.14.1 release"
  • Loading branch information
mvieth committed May 4, 2024
2 parents 7597244 + 38ab31e commit 739229a
Show file tree
Hide file tree
Showing 3 changed files with 10 additions and 4 deletions.
4 changes: 2 additions & 2 deletions io/src/obj_io.cpp
Expand Up @@ -711,8 +711,8 @@ pcl::OBJReader::read (const std::string &file_name, pcl::TextureMesh &mesh,
// Tokenize the line
pcl::split (st, line, "\t\r ");

// Ignore comments
if (st[0] == "#")
// Ignore comments and lines with only whitespace
if (st.empty() || st[0] == "#")
continue;
// Vertex
if (st[0] == "v")
Expand Down
8 changes: 7 additions & 1 deletion io/src/pcd_io.cpp
Expand Up @@ -677,7 +677,13 @@ pcl::PCDReader::read (const std::string &file_name, pcl::PCLPointCloud2 &cloud,
pcl::console::TicToc tt;
tt.tic ();

if (file_name.empty () || !pcl_fs::exists (file_name))
if (file_name.empty ())
{
PCL_ERROR ("[pcl::PCDReader::read] No file name given!\n");
return (-1);
}

if (!pcl_fs::exists (file_name))
{
PCL_ERROR ("[pcl::PCDReader::read] Could not find file '%s'.\n", file_name.c_str ());
return (-1);
Expand Down
Expand Up @@ -211,7 +211,7 @@ TransformationEstimationSVD<PointSource, PointTarget, Scalar>::
Eigen::Matrix<Scalar, 3, 3> R = v * u.transpose();

// Return the correct transformation
transformation_matrix.topLeftCorner(3, 3) = R;
transformation_matrix.template topLeftCorner<3, 3>() = R;
const Eigen::Matrix<Scalar, 3, 1> Rc(R * centroid_src.template head<3>());
transformation_matrix.template block<3, 1>(0, 3) =
centroid_tgt.template head<3>() - Rc;
Expand Down

0 comments on commit 739229a

Please sign in to comment.