From: Steve Holme Date: Sun, 30 Nov 2014 14:48:01 +0000 (+0000) Subject: transfer: Code style policing X-Git-Tag: curl-7_40_0~277 X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=56120ca04bc6c0dc4e942504c7abf018b7ef8428;p=curl transfer: Code style policing Prefer ! rather than NULL in if statements, added comments and updated function spacing, argument spacing and line spacing to be more readble. --- diff --git a/lib/transfer.c b/lib/transfer.c index 2cc7f7881..a7ab08ecf 100644 --- a/lib/transfer.c +++ b/lib/transfer.c @@ -910,13 +910,16 @@ static CURLcode readwrite_upload(struct SessionHandle *data, (data->set.prefer_ascii) || #endif (data->set.crlf))) { - if(data->state.scratch == NULL) { - data->state.scratch = malloc(2*BUFSIZE); - if(data->state.scratch == NULL) { - failf (data, "Failed to alloc scratch buffer!"); + /* Do we need to allocate a scratch buffer? */ + if(!data->state.scratch) { + data->state.scratch = malloc(2 * BUFSIZE); + if(!data->state.scratch) { + failf(data, "Failed to alloc scratch buffer!"); + return CURLE_OUT_OF_MEMORY; } } + /* * ASCII/EBCDIC Note: This is presumably a text (not binary) * transfer so the data should already be in ASCII.