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

[VT]: Fix missing lock guard and deadlock in iterator #453

Merged
merged 1 commit into from
Mar 15, 2024

Conversation

GwnDaan
Copy link
Member

@GwnDaan GwnDaan commented Mar 15, 2024

Describe your changes

When testing I found that the seeder example gave a deadlock when trying to spam the settings soft-key immediately after startup. This PR resolves that issue.

Might be a fix for #449, further testing needed

@GwnDaan GwnDaan added bug Something isn't working iso: virtual terminal Related to the ISO-11783:7 standard labels Mar 15, 2024
@GwnDaan GwnDaan requested a review from ad3154 March 15, 2024 22:05
@GwnDaan GwnDaan self-assigned this Mar 15, 2024
Copy link

@GwnDaan GwnDaan merged commit f192b2f into main Mar 15, 2024
10 checks passed
@GwnDaan GwnDaan deleted the daan/fix-command-queue-iterator branch March 15, 2024 22:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working iso: virtual terminal Related to the ISO-11783:7 standard
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants