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

Jump to bottom

c-hyper: remove the hyper_executor_poll() loop from Curl_http #7499

Closed
wants to merge 3 commits into from
Closed

c-hyper: remove the hyper_executor_poll() loop from Curl_http #7499

wants to merge 3 commits into from

Conversation

bagder
Copy link
Member

@bagder bagder commented Jul 26, 2021

  1. it's superfluous
  2. it didn't work identically to the Curl_hyper_stream one which could
    cause problems like Race condition in c-hyper.c, Curl_http() #7486

Pointed-out-by: David Cook

@bagder
c-hyper: remove the hyper_executor_poll() loop from Curl_http
fb80b62 
1. it's superfluous
2. it didn't work identically to the Curl_hyper_stream one which could
   cause problems like #7486

Pointed-out-by: David Cook
@bagder bagder added the Hyper label Jul 26, 2021
@bagder bagder mentioned this pull request Jul 26, 2021
Closed
@bagder

This comment has been minimized.

Sign in to view
@bagder bagder closed this in 2c688a4 Jul 27, 2021
@bagder bagder deleted the bagder/hyper-rm-exec-poll branch July 27, 2021 08:43
@bagder bagder mentioned this pull request Jul 27, 2021
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
Hyper
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
1 participant
@bagder