Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

CopyPointCloud #522

Closed
mhoedlmo opened this issue Feb 20, 2014 · 4 comments
Closed

CopyPointCloud #522

mhoedlmo opened this issue Feb 20, 2014 · 4 comments

Comments

@mhoedlmo
Copy link

When building Kinfu, the compiler complains at pcl::CopyPointCloud: "a class cannot be its own base class". Solution:

diff --git a/common/include/pcl/common/impl/io.hpp b/common/include/pcl/common/impl/io.hpp 
index 4fa068e..046e8b6 100644 
--- a/common/include/pcl/common/impl/io.hpp 
+++ b/common/include/pcl/common/impl/io.hpp 
@@ -126,7 +126,7 @@ pcl::copyPointCloud (const pcl::PointCloud<PointInT> &cloud_in, 
   else 
     // Iterate over each point 
     for (size_t i = 0; i < cloud_in.points.size (); ++i) 
-      copyPoint (cloud_in.points[i], cloud_out.points[i]); 
+    copyPoint<PointInT, PointOutT>(cloud_in.points[i], cloud_out.points[i]); 
 }
@jspricke
Copy link
Member

Hi mhoedlmo,

great find. Can you make this a pull requst? Don't hesitate to ask, if
you need help with that.

Cheers Jochen

@mhoedlmo
Copy link
Author

Hi jspricke,

I don't have the repository on this machine and if I understood it right, I need to create a local branch in order to make a pull request. I would be happy if you can make the pull request for me!

Cheers

@jspricke
Copy link
Member

Should be possible to do it on the website, just klick fork, then edit
the file and klick pull request. If you do it, you will be listed as the
author ;).

@jspricke
Copy link
Member

Should be fixed with #593.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants