On 22.04.2015 21:30, Rainer Jung wrote: > Am 22.04.2015 um 17:49 schrieb Kaspar Brand: >> Thanks. In the patch for ssl_private.h, the complete NPN block should >> actually be dropped - the same block is are already part of >> ssl_private.h, just 10 lines above. > > I've kept the new one and dropped the old one in r1675459.
It's nitpicking, but... the old one was already correct, actually. All this stuff is in a larger "#if !defined(OPENSSL_NO_TLSEXT) ..." block, so there's no need to repeat NO_TLSEXT. I'd rather get consensus on completely dropping HAVE_TLS_NPN from trunk anyway, though, so it's really a very minor point right now ;-) Kaspar