diff --git a/CMake/PickyWarnings.cmake b/CMake/PickyWarnings.cmake index e86e184bfa..2785c7d24b 100644 --- a/CMake/PickyWarnings.cmake +++ b/CMake/PickyWarnings.cmake @@ -280,6 +280,24 @@ if(PICKY_COMPILER) list(APPEND _picky "-Wno-conversion") # Avoid false positives endif() endif() + elseif(MSVC AND MSVC_VERSION LESS_EQUAL 1943) # Skip for untested/unreleased newer versions + list(APPEND _picky "-Wall") + list(APPEND _picky "-wd4061") # enumerator 'A' in switch of enum 'B' is not explicitly handled by a case label + list(APPEND _picky "-wd4191") # 'type cast': unsafe conversion from 'FARPROC' to 'void (__cdecl *)(void)' + list(APPEND _picky "-wd4255") # no function prototype given: converting '()' to '(void)' (in winuser.h) + list(APPEND _picky "-wd4464") # relative include path contains '..' + list(APPEND _picky "-wd4548") # expression before comma has no effect; expected expression with side-effect (in FD_SET()) + list(APPEND _picky "-wd4574") # 'M' is defined to be '0': did you mean to use '#if M'? (in ws2tcpip.h) + list(APPEND _picky "-wd4668") # 'M' is not defined as a preprocessor macro, replacing with '0' for '#if/#elif' (in winbase.h) + list(APPEND _picky "-wd4710") # 'snprintf': function not inlined + list(APPEND _picky "-wd4711") # function 'A' selected for automatic inline expansion + list(APPEND _picky "-wd4746") # volatile access of '' is subject to /volatile: setting; + # consider using __iso_volatile_load/store intrinsic functions (ARM64) + list(APPEND _picky "-wd4774") # 'snprintf': format string expected in argument 3 is not a string literal + list(APPEND _picky "-wd4820") # 'A': 'N' bytes padding added after data member 'B' + if(MSVC_VERSION GREATER_EQUAL 1900) + list(APPEND _picky "-wd5045") # Compiler will insert Spectre mitigation for memory load if /Qspectre switch specified + endif() endif() endif() diff --git a/lib/dynhds.h b/lib/dynhds.h index fb162a30de..191acdd1c2 100644 --- a/lib/dynhds.h +++ b/lib/dynhds.h @@ -156,14 +156,14 @@ CURLcode Curl_dynhds_cadd(struct dynhds *dynhds, /** * Add a single header from an HTTP/1.1 formatted line at the end. Line - * may contain a delimiting \r\n or just \n. Any characters after + * may contain a delimiting CRLF or just LF. Any characters after * that will be ignored. */ CURLcode Curl_dynhds_h1_cadd_line(struct dynhds *dynhds, const char *line); /** * Add a single header from an HTTP/1.1 formatted line at the end. Line - * may contain a delimiting \r\n or just \n. Any characters after + * may contain a delimiting CRLF or just LF. Any characters after * that will be ignored. */ CURLcode Curl_dynhds_h1_add_line(struct dynhds *dynhds, diff --git a/lib/ftp.c b/lib/ftp.c index 393db0b4c8..f7d756a2a9 100644 --- a/lib/ftp.c +++ b/lib/ftp.c @@ -330,8 +330,7 @@ static void freedirs(struct ftp_conn *ftpc) } #ifdef CURL_PREFER_LF_LINEENDS -/*********************************************************************** - * +/* * Lineend Conversions * On ASCII transfers, e.g. directory listings, we might get lines * ending in '\r\n' and we prefer just '\n'.