Re: [PATCH] Break if is_empty_data is true at the end of the while loop

2014-08-03 Thread Daniel Stenberg
On Sun, 3 Aug 2014, Tatsuhiro Tsujikawa wrote: Your patch works fine for me. Patch attached based on yours. Thanks, merged! -- / daniel.haxx.se --- List admin: http://cool.haxx.se/list/listinfo/curl-library Etiquette: http://

Re: [PATCH] Break if is_empty_data is true at the end of the while loop

2014-08-02 Thread Tatsuhiro Tsujikawa
On Sun, Aug 3, 2014 at 6:37 AM, Daniel Stenberg wrote: > On Sat, 2 Aug 2014, Tatsuhiro Tsujikawa wrote: > > This patch adds break statement if is_empty_data is true at the end of >> while loop in readwrite_data. Without this break statement, CPU usage goes >> to 100% if HTTP/2 transfer is interr

Re: [PATCH] Break if is_empty_data is true at the end of the while loop

2014-08-02 Thread Daniel Stenberg
On Sat, 2 Aug 2014, Tatsuhiro Tsujikawa wrote: This patch adds break statement if is_empty_data is true at the end of while loop in readwrite_data. Without this break statement, CPU usage goes to 100% if HTTP/2 transfer is interrupted (RST_STREAM) and its recv function keeps return 0. I don'

[PATCH] Break if is_empty_data is true at the end of the while loop

2014-08-01 Thread Tatsuhiro Tsujikawa
This patch adds break statement if is_empty_data is true at the end of while loop in readwrite_data. Without this break statement, CPU usage goes to 100% if HTTP/2 transfer is interrupted (RST_STREAM) and its recv function keeps return 0. I'm not really sure this patch does the right thing. Fro th