Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

[C] Fix mutexes on Windows #867

Merged
merged 1 commit into from
Feb 20, 2020
Merged

[C] Fix mutexes on Windows #867

merged 1 commit into from
Feb 20, 2020

Conversation

ltrzesniewski
Copy link
Contributor

  • Lock/unlock did not work as an indirection was missing
  • Made the function prototypes match the Linux version
  • Added aeron_mutex_destroy

Part of #863

@ltrzesniewski
[C] Fix mutexes on Windows
e9494db 
 - Lock/unlock did not work as an indirection was missing
 - Made the function prototypes match the Linux version
 - Added aeron_mutex_destroy
@ltrzesniewski
Copy link
Contributor Author

There were some additional whitespace changes in aeron_thread.h in the original PR (I removed the empty lines between aeron_thread_* functions) which are not included here to avoid a merge conflict with #864.

@tmontgomery tmontgomery merged commit 326ea6c into aeron-io:master Feb 20, 2020
@ltrzesniewski ltrzesniewski deleted the winfix-mutex branch February 20, 2020 16:03
# for free to join this conversation on GitHub. Already have an account? # to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@ltrzesniewski @tmontgomery