This is a MSYS/MinGW-only warning; full warning text is: Use of uninitialized value in numeric gt (>) at ../../curl/tests/runtests.pl line 2227.
In many states the easy_conn pointer is referenced and just assumed to be working. This is an added extra check since analyzing indicates there's a risk we can end up in these states with a NULL pointer there.
va_end() needs to be used after va_start() and we don't normally use Curl_ prefixes for purely static functions.
Two commits ago, we fixed a bug where the connction would be closed prematurely after a HEAD. Now I added connection-monitor to test 48 and added a second HEAD and make sure that both are sent over the same connection. This triggered a failure before the bug fix and now works. Will help us avoid a future regression of this kind.
This makes verifying easier and makes us more sure curl closes the connection only at the correct point in time. Adjusted test 206 and 1008 accordingly and updated the docs for it.
A HEAD response has no body length and gets the headers like the corresponding GET would so it should not get closed after the response based on the same rules. This mistake caused connections that did HEAD to get closed too often without a valid reason. Bug: http://curl.haxx.se/bug/view.cgi?id=3542731 Reported by: Eelco Dolstra
1 - str2offset() no longer accepts negative numbers since offsets are by nature positive. 2 - introduced str2unum() for the command line parser that accepts numericals which are not supposed to be negative, so that it will properly complain on apparent bad uses and mistakes. Bug: http://curl.haxx.se/mail/archive-2012-07/0013.html