calloc and ai_addrlen expect different (usually unsigned) types.

Suggested-by: Dan Fandrich

... instead of numerical order.

Closes #1466

assign string literals to const char * instead of char * in order to
avoid a lot of these warnings:
cast from 'const char *' to 'char *' drops const qualifier
[-Wcast-qual]

Windows does not allow setting the locale with environment variables (as
the test attempted to do), so the test failed when run with a user
locale that has a comma as radixchar. Changed the test to call
setlocale() explicitly to ensure that a known working locale is set even
on Windows.

SEND_QUAL_ARG2 had to be set, but was never used. Use it in swrite to
avoid warnings about casting away low-level const.

Closes https://github.com/curl/curl/pull/1464

Fix the following warnings when building the tests by using the correct
types:
cast from 'const char *' to 'void *' drops const qualifier
[-Wcast-qual]
implicit conversion changes signedness [-Wsign-conversion]

Closes https://github.com/curl/curl/pull/1452

Closes https://github.com/curl/curl/pull/1452

Previous TODO wanting to write in chunks. We should support writing more
at once since some TELNET servers may respond immediately upon first
byte written such as WHOIS servers.

Closes #1389

... and instead properly respond with an error message to the user
instead of silently ignoring.

Fixes #1453
Closes #1458

Support for _MPRINTF_REPLACE in mprintf.h was removed in
55452ebd, replaced with curl_printf.h.

Closes #1449

The buffer can have other sizes.

... to properly use the dynamically set buffer size!

It was a wrong assumption that it could do that!

To make it suitably independent of the receive buffer and its flexible
size.

Removes the need for CURL_BUFSIZE

... instead of clobbering the download buffer.

The buffer is needed to receive FTP, HTTP CONNECT responses etc so
already at this size things risk breaking and smaller is certainly not
wise.