Skip to content
This repository has been archived by the owner on Nov 1, 2023. It is now read-only.

Ignore benign recv hangup in timer #2722

Merged
merged 1 commit into from
Dec 22, 2022
Merged

Conversation

ranweiler
Copy link
Member

Remove a stray unwrap. There's no reason to panic the thread, since the timeout and worker function are inherently racy.

@codecov-commenter
Copy link

codecov-commenter commented Dec 22, 2022

Codecov Report

Merging #2722 (bdbd792) into main (07a0a92) will not change coverage.
The diff coverage is 0.00%.

@@           Coverage Diff           @@
##             main    #2722   +/-   ##
=======================================
  Coverage   28.74%   28.74%           
=======================================
  Files         320      320           
  Lines       38144    38144           
=======================================
  Hits        10963    10963           
  Misses      27181    27181           
Impacted Files Coverage Δ
src/agent/coverage/src/timer.rs 0.00% <0.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@ranweiler ranweiler merged commit d51d4e0 into microsoft:main Dec 22, 2022
@ranweiler ranweiler deleted the ignore-hangup branch December 22, 2022 18:02
@AdamL-Microsoft AdamL-Microsoft mentioned this pull request Jan 5, 2023
@ghost ghost locked as resolved and limited conversation to collaborators Jan 22, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants