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

fix #1180: race in startStateLoop #1864

Merged
merged 1 commit into from
Mar 6, 2015
Merged

fix #1180: race in startStateLoop #1864

merged 1 commit into from
Mar 6, 2015

Conversation

dgnorton
Copy link
Contributor

@dgnorton dgnorton commented Mar 6, 2015

No description provided.

@dgnorton dgnorton force-pushed the fix-1180 branch 2 times, most recently from e49d1f9 to 3415a7e Compare March 6, 2015 15:07
defer l.wg.Done()

l.Logger.Printf("log state change: %s => %s", l.state, state)
Copy link
Contributor

Choose a reason for hiding this comment

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

With these two lines in place, you should remove them from the for loop, right?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

It still needs them so it can log the transition each time through the loop.

@toddboom
Copy link
Contributor

toddboom commented Mar 6, 2015

same question as philip above, otherwise looks good to me.

dgnorton added a commit that referenced this pull request Mar 6, 2015
@dgnorton dgnorton merged commit 3454523 into master Mar 6, 2015
@dgnorton dgnorton deleted the fix-1180 branch March 6, 2015 18:21
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.

3 participants