curl / Mailing Lists / curl-library / Single Mail
Buy commercial curl support from WolfSSL. We help you work out your issues, debug your libcurl applications, use the API, port to new platforms, add new features and more. With a team lead by the curl founder himself.

Re: Potential confusion in http_proxy.c and a recommendation

From: Daniel Stenberg via curl-library <>
Date: Mon, 9 Nov 2020 23:51:20 +0100 (CET)

On Mon, 9 Nov 2020, Hasabnis, Niranjan via curl-library wrote:

> We believe that using “if (s->keepon > 1)” would eliminate this confusion
> and capture the intended semantics precisely.

I think you've pointed out code that could be written clearer, yes. But I
think an even better improvement to this logic would be to use an enum or
defined values that include all three used values as state names.

What do you think about my proposal over at:

Received on 2020-11-09