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

Fix race condition on sending in full MessageQueue #760

Merged
merged 1 commit into from Aug 2, 2014

Conversation

Projects
None yet
2 participants
@villytiger
Contributor

villytiger commented Aug 2, 2014

There was a race condition in MessageQueue if one task waits for a message in receive method and other task sends so many messages that queue becomes full, and then it locks on sending becuase first task didnt notify.

@s-ludwig

This comment has been minimized.

Show comment
Hide comment
@s-ludwig

s-ludwig Aug 2, 2014

Member

Thanks! Good catch.

Member

s-ludwig commented Aug 2, 2014

Thanks! Good catch.

s-ludwig added a commit that referenced this pull request Aug 2, 2014

Merge pull request #760 from villytiger/race-condition
Fix race condition on sending in full MessageQueue

@s-ludwig s-ludwig merged commit 2fcba43 into vibe-d:master Aug 2, 2014

1 check passed

continuous-integration/travis-ci The Travis CI build passed
Details

@villytiger villytiger deleted the villytiger:race-condition branch Nov 28, 2015

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment