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

scheduler: Fix regression in background thread (Issue #934) #937

Conversation

andreas-karlson
Copy link

The con->bg_pending value is used to indicate that the background thread has finished running and that the connection can be safely closed.

This commit makes the con->bg_pending = 0 only after the connection can be safely closed.

The con->bg_pending value is used to indicate that the background thread
has finished running and that the connection can be safely closed.

This commit makes the con->bg_pending = 0 only after the connection
can be safely closed.
Copy link
Member

@michaelrsweet michaelrsweet left a comment

Choose a reason for hiding this comment

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

+1

@michaelrsweet michaelrsweet self-assigned this Apr 10, 2024
@michaelrsweet michaelrsweet added bug Something isn't working priority-high labels Apr 10, 2024
@michaelrsweet michaelrsweet added this to the v2.4.x milestone Apr 10, 2024
@michaelrsweet michaelrsweet merged commit ac809be into OpenPrinting:2.4.x Apr 10, 2024
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working priority-high
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants