cURL / Mailing Lists / curl-library / Single Mail


Re: polarssl: add ALPN support

From: Fabian Frank <>
Date: Thu, 22 May 2014 10:15:37 -0700

On May 21, 2014, at 11:38 PM, Dan Fandrich <> wrote:
> You can also eliminate a #ifdef block by moving the definition of this
> into the existing #ifdef block where it's being used.
> +#ifdef HAS_ALPN
> + const char* next_protocol;
> +#endif
> And the same with this one. The fewer #ifdef blocks, the easier to read.

Previously people pointed out that variable definitions have to be at the top of functions in curl code, which is why I have done it this way. Before that I had them inside the ifdef blocks, as you suggested. I am ok with either one, as long as itís clear which one we want. :-)


List admin:
Received on 2014-05-22