pcl icon indicating copy to clipboard operation
pcl copied to clipboard

Added PLYWriter::writeBinary() ofstream overload

Open kaipomauli opened this issue 1 year ago • 1 comments
trafficstars

modified existing PLYWriter::writeBinary() to open file and then call new overload for data aggregation

added PLYWriter::writeBinary() overload that adds data to referenced ofstream buffer

kaipomauli avatar Feb 18 '24 21:02 kaipomauli

By the way (in case you did not know this): to change a pull request, you can push more commits to the branch, no need to close the pull request and open a new one :slightly_smiling_face: And for the future, I would recommend to always create a new branch for a pull request, instead of using the master branch

mvieth avatar Feb 21 '24 12:02 mvieth

Hey, I assumed one of the tests that read ply files would catch a discrepancy in formating so I did not dig that much further XD Awesome I'll get onto another issue.

kaipomauli avatar Feb 27 '24 16:02 kaipomauli