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

Remove Boost Thread and Chrono #1396

Merged
merged 1 commit into from
Jul 25, 2022
Merged

Remove Boost Thread and Chrono #1396

merged 1 commit into from
Jul 25, 2022

Conversation

Bushstar
Copy link
Member

  • Remove boost::chrono and boost::sleep

  • Remove Boost mutex

  • Remove Boost thread_group

  • Remove Boost thread/mutex from SPV

  • Remove Boost interruption_point

  • Replace Boost shared_mutex with C++17 std::shared_mutex

  • Remove Boost Chrono and Thread from build system

  • Add missing include

  • Remove duplicate start of scheduler thread

  • Remove Boost case_conv

  • Remove Boost join and transformed

  • Do not interrupt ForEach loop

  • Add notes to interrupt replacements

Co-authored-by: Prasanna Loganathar [email protected]

What kind of PR is this?:

/kind

What this PR does / why we need it:

Which issue(s) does this PR fixes?:

Fixes #

Additional comments?:

* Remove boost::chrono and boost::sleep

* Remove Boost mutex

* Remove Boost thread_group

* Remove Boost thread/mutex from SPV

* Remove Boost interruption_point

* Replace Boost shared_mutex with C++17 std::shared_mutex

* Remove Boost Chrono and Thread from build system

* Add missing include

* Remove duplicate start of scheduler thread

* Remove Boost case_conv

* Remove Boost join and transformed

* Do not interrupt ForEach loop

* Add notes to interrupt replacements

Co-authored-by: Prasanna Loganathar <[email protected]>
@prasannavl prasannavl merged commit 52c8526 into master Jul 25, 2022
@prasannavl prasannavl deleted the remove-thread-again branch July 25, 2022 10:29
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

Successfully merging this pull request may close these issues.

2 participants