On Mon, May 19, 2014 at 02:20:25AM -0700, Fabian Frank wrote:
> here is a patch to add ALPN support when curl is being used with PolarSSL.
> Please note that nghttp2’s public test servers have ALPN disabled right now,
> so you have to run your own nghttpd locally to test ALPN negotiation of h2-12.
> +#ifdef HAS_ALPN
> + const char* protocols[] = {NGHTTP2_PROTO_VERSION_ID, ALPN_HTTP_1_1,
> + NULL};
> +#endif
Minor nitpick: this should be static const char* const protocols[] to make the
array itself const and avoid building it on the stack each time.
>>> Dan
-------------------------------------------------------------------
List admin: http://cool.haxx.se/list/listinfo/curl-library
Etiquette: http://curl.haxx.se/mail/etiquette.html
