cURL / Mailing Lists / curl-library / Single Mail

curl-library

Re: snprintf in http2.c

From: Daniel Stenberg <daniel_at_haxx.se>
Date: Fri, 6 Sep 2013 13:59:03 +0200 (CEST)

On Fri, 6 Sep 2013, Gisle Vanem wrote:

> I built NgHTTP2 some days ago using MSVC 2010. I was amazed of how easy it
> was. Not very Unix/Posix centric at all. But using the first little merge of
> nghttp2 into libcurl, I stumbeled on the missing 'snprintf' in MSVCRT. Isn't
> this how we do it for other libcurl files? I.e. use 'curl_msnprintf' and not
> 'snprintf' directly:

Right, thanks. This is also why I like to push things like this early, since
there are always a lot of these details that I tend to miss...

I moved the include line to below the #ifdef USE_NGHTTP2 to prevent extra
processing time when not building with http2 support enabled.

Fix pushed!

-- 
  / daniel.haxx.se
-------------------------------------------------------------------
List admin: http://cool.haxx.se/list/listinfo/curl-library
Etiquette:  http://curl.haxx.se/mail/etiquette.html
Received on 2013-09-06