Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

add an internal queue to signal that a datagram frame has been dequeued #3081

Merged
merged 1 commit into from
Mar 10, 2021

Conversation

mengelbart
Copy link
Contributor

Fixes #3075

@codecov
Copy link

codecov bot commented Mar 9, 2021

Codecov Report

Merging #3081 (bcc7baf) into master (eb3e100) will decrease coverage by 0.01%.
The diff coverage is 75.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #3081      +/-   ##
==========================================
- Coverage   86.04%   86.03%   -0.01%     
==========================================
  Files         132      132              
  Lines        9435     9441       +6     
==========================================
+ Hits         8118     8122       +4     
- Misses        955      957       +2     
  Partials      362      362              
Impacted Files Coverage Δ
datagram_queue.go 88.89% <75.00%> (-4.44%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update eb3e100...bcc7baf. Read the comment docs.

@marten-seemann marten-seemann self-requested a review March 10, 2021 04:04
@marten-seemann
Copy link
Member

Thank you @mengelbart!

@marten-seemann marten-seemann merged commit 20018c8 into quic-go:master Mar 10, 2021
@aschmahmann aschmahmann mentioned this pull request May 14, 2021
71 tasks
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Deadlock in datagramQueue
2 participants