Skip to content

Conversation

@ericmutta
Copy link
Contributor

Move bracketed comment (to be moved, the thread must be at the head of the waiting queue) towards the end of the sentence so it doesn't interrupt the reading flow when the sentence refers to the Pulse and PulseAll methods. I have also clarified that the thread in question is the one that called the Wait method, which helps explain why it would be in the waiting queue and would later be moved to the ready queue when Pulse or PulseAll is called.

Move bracketed comment `(to be moved, the thread must be at the head of the waiting queue)` towards the end of the sentence so it doesn't interrupt the reading flow when the sentence refers to the `Pulse` and `PulseAll` methods. I have also clarified that the thread in question is the one that called the `Wait` method, which helps explain why it would be in the waiting queue and would later be moved to the ready queue when `Pulse` or `PulseAll` are called.
@ericmutta
Copy link
Contributor Author

Is anyone monitoring these PRs? This one has been idle for 13 days now!

@ericmutta
Copy link
Contributor Author

Pinging @mairaw

Copy link

@rpetrusha rpetrusha left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm sorry that this has taken us so long, @ericmutta. Your changes look good, so I'll merge your PR now. The changes should be live on docs.microsoft.com in the next day or two.

@rpetrusha rpetrusha merged commit fde80fc into dotnet:master Jul 29, 2019
@ericmutta
Copy link
Contributor Author

@rpetrusha many thanks for following up!

@ericmutta ericmutta deleted the patch-1 branch July 30, 2019 13:51
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