search

valette / ACVD

Fast simplification of 3D surface meshes
https://www.creatis.insa-lyon.fr/~valette/public/project/acvd/
Other
266 stars 57 forks source link

fix: replace vtk mutexx with std::mutex #42

Closed kayarre closed 3 years ago

kayarre commented 3 years ago

tried replacing the vtk mutex with std::mutex. hooray for c++11. Compiles, needs testing.

kayarre commented 3 years ago

I tested it locally on the files listed in the README and it appears to work.

valette commented 3 years ago

Actually this code is not used if the USE_MULTITHREADING advanced option in cmake is not used. It needed small fixes because the semantics with try_lock were different, but it works now with the option USE_MULTITHREADING. Pulled, thanks!

kayarre commented 3 years ago

Ahh totally missed the multi-threading option. Thank you!