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

csync: using std::thread; #507

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

csync: using std::thread; #507

wants to merge 1 commit into from

Conversation

piedar
Copy link
Contributor

@piedar piedar commented May 31, 2017

  • No API changes!
  • Eliminates dependency on pthreads-win32.
  • Takes advantage of thread and mutex from C++11.
  • One downside: the c_sync library is 15 kB bigger.

todo

  • Should C++ globals be static? Appears yes.
  • using aliases might be premature abstraction.
  • Examples still depend on pthreads-win32.
  • Old binaries work on new library, but there might be pitfalls on other platforms.

@piedar piedar added this to the v0.6.0 milestone May 31, 2017
@piedar piedar force-pushed the std-thread branch 2 times, most recently from 5abe51f to 51b7cbe Compare June 1, 2017 06:22
Signed-off-by: Benn Snyder <benn.snyder@gmail.com>
@piedar piedar changed the title c_sync: using std::thread; using std::thread; Jun 1, 2017
@piedar piedar changed the title using std::thread; csync: using std::thread; Jun 1, 2017
@piedar piedar modified the milestones: v0.6.0, v0.6.1 Mar 23, 2018
@piedar piedar modified the milestones: v0.6.1, v0.6.3, v0.6.2 May 10, 2020
@piedar piedar modified the milestones: v0.6.2, v0.6.3 Feb 11, 2021
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.

None yet

1 participant