Removing cosim::utility::shared_mutex

open-simulation-platform / libcosim

OSP C++ co-simulation library

https://open-simulation-platform.github.io/libcosim

Mozilla Public License 2.0

61 stars 10 forks source link

Removing cosim::utility::shared_mutex #692

Closed ljamt closed 2 years ago

ljamt commented 2 years ago

Removing the fiber friendly cosim::utility::shared_mutex
Also bumping version as this has not been done after 0.8.3 release

restenb commented 2 years ago

Looks good. With the removal of fibers and swap to std::mutex, are there now any changes that are necessary or beneficial to make to cosim::utility::file_lock as well?

I don't think any file operations are currently being done on the thread pool.

kyllingstad commented 2 years ago

I've pushed some doc updates now. I also took the liberty of removing some final traces of fibers in a couple of other places.

ljamt commented 2 years ago

I've pushed some doc updates now. I also took the liberty of removing some final traces of fibers in a couple of other places.

Great! Thanks a lot :)