From: Viktor Szakats Date: Mon, 23 Dec 2024 17:05:40 +0000 (+0100) Subject: build: drop `-Winline` picky warning X-Git-Tag: curl-8_12_0~278 X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=c80c399c45cc8d0a323e4136b166b245b3492532;p=thirdparty%2Fcurl.git build: drop `-Winline` picky warning To avoid this in certain debug build cases: ``` ./lib/easy_lock.h:72:20: error: inlining failed in call to 'curl_simple_lock_lock': function not considered for inlining [-Werror=inline] ``` Ref: #13596 Reported-by: Rudi Heitbaum Fixes #15815 Closes #15819 --- diff --git a/CMake/PickyWarnings.cmake b/CMake/PickyWarnings.cmake index bfafbb970a..c10b408ffb 100644 --- a/CMake/PickyWarnings.cmake +++ b/CMake/PickyWarnings.cmake @@ -75,7 +75,6 @@ if(PICKY_COMPILER) list(APPEND _picky_enable -Wbad-function-cast # clang 2.7 gcc 2.95 -Wconversion # clang 2.7 gcc 2.95 - -Winline # clang 1.0 gcc 1.0 -Wmissing-declarations # clang 1.0 gcc 2.7 -Wmissing-prototypes # clang 1.0 gcc 1.0 -Wnested-externs # clang 1.0 gcc 2.7 diff --git a/m4/curl-compilers.m4 b/m4/curl-compilers.m4 index 787423d1ff..b3521f4be8 100644 --- a/m4/curl-compilers.m4 +++ b/m4/curl-compilers.m4 @@ -797,7 +797,7 @@ AC_DEFUN([CURL_SET_COMPILER_WARNING_OPTS], [ CURL_ADD_COMPILER_WARNINGS([tmp_CFLAGS], [all extra]) CURL_ADD_COMPILER_WARNINGS([tmp_CFLAGS], [pointer-arith write-strings]) CURL_ADD_COMPILER_WARNINGS([tmp_CFLAGS], [shadow]) - CURL_ADD_COMPILER_WARNINGS([tmp_CFLAGS], [inline nested-externs]) + CURL_ADD_COMPILER_WARNINGS([tmp_CFLAGS], [nested-externs]) CURL_ADD_COMPILER_WARNINGS([tmp_CFLAGS], [missing-declarations]) CURL_ADD_COMPILER_WARNINGS([tmp_CFLAGS], [missing-prototypes]) tmp_CFLAGS="$tmp_CFLAGS -Wno-long-long" @@ -943,7 +943,7 @@ AC_DEFUN([CURL_SET_COMPILER_WARNING_OPTS], [ # dnl Only gcc 2.7 or later if test "$compiler_num" -ge "207"; then - CURL_ADD_COMPILER_WARNINGS([tmp_CFLAGS], [inline nested-externs]) + CURL_ADD_COMPILER_WARNINGS([tmp_CFLAGS], [nested-externs]) dnl If not cross-compiling with a gcc older than 3.0 if test "x$cross_compiling" != "xyes" || test "$compiler_num" -ge "300"; then