wget-dev
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: wget2 | * configure.ac: Add -Wno-declaration-after-statement to clan


From: Avinash Sonawane (@rootkea)
Subject: Re: wget2 | * configure.ac: Add -Wno-declaration-after-statement to clang-15 manywarnings (660b3131)
Date: Wed, 22 Jun 2022 13:01:44 +0000



Avinash Sonawane started a new discussion on configure.ac: 
https://gitlab.com/gnuwget/wget2/-/commit/660b31311865a014f10caeff845146f2c504a64d#note_1000826845

>        WARN_CFLAGS="$WARN_CFLAGS -Wno-reserved-identifier"
>      fi
>  
> +    if test $CLANG_VERSION -ge 15; then

Shouldn't this be `14`?

Also, maybe we shouldn't use `-Weverything` for clang? As per some `clang` devs 
that option is for testing clang itself (to test all possible warnings). There 
are even some conflicting warnings in `-Weverything`. And so just `-Wall 
-Wextra` should suffice instead?

https://quuxplusone.github.io/blog/2018/12/06/dont-use-weverything/

-- 
Reply to this email directly or view it on GitLab: 
https://gitlab.com/gnuwget/wget2/-/commit/660b31311865a014f10caeff845146f2c504a64d#note_1000826845
You're receiving this email because of your account on gitlab.com.




reply via email to

[Prev in Thread] Current Thread [Next in Thread]