[vlc-devel] [PATCH] configure: fail on casting incompatible pointers

[Rémi Denis-Courmont]

So you take responsibility for guaranteeing that all headers of all underlying libraries and SDK will never cause cast warnings?

We already discussed this, and everybody except Pierre d'H. was against, for good reasons.

Le 2 octobre 2019 14:03:32 GMT+03:00, Steve Lhomme <robux4 at ycbcr.xyz> a écrit :
>On 2019-10-02 13:02, Rémi Denis-Courmont wrote:
>> Yes and last I checked, VLC includes libavutil header files.
>
>I do include libavutil and have no problem with this compiler flag.
>
>> Le 2 octobre 2019 13:27:38 GMT+03:00, Steve Lhomme <robux4 at ycbcr.xyz>
>a 
>> écrit :
>> 
>>     On 2019-10-02 12:20, Rémi Denis-Courmont wrote:
>> 
>>         And VLC depends on some libraries that rely on bad casts,
>notably
>>         libavutil (heap helpers).
>> 
>> 
>>     This is the configure script of VLC, not the contribs. I use this
>flag
>>     all day long on Win32 and Linux builds with no issue. That
>enables
>>     spotting errors like the AMT one very quickly.
>> 
>>         Le 2 octobre 2019 13:18:30 GMT+03:00, "Rémi Denis-Courmont"
>>         <remi at remlab.net> a écrit :
>> 
>>         Next people will add incorrect manual casts to silence the
>errors
>>         instead of fixing them properly. Not a good idea.
>> 
>>         Le 2 octobre 2019 13:14:14 GMT+03:00, Romain Vimont
>>         <rom1v at videolabs.io> a écrit :
>>        
>------------------------------------------------------------------------
>>         configure.ac | 2 +-
>>         1 file changed, 1 insertion(+), 1 deletion(-)
>> 
>>         diff --git a/configure.ac b/configure.ac
>>         index 210b4ca537..2fa70daf22 100644
>>         --- a/configure.ac
>>         +++ b/configure.ac
>>         @@ -988,7 +988,7 @@ dnl
>>         AX_APPEND_COMPILE_FLAGS([-Wall -Wextra -Wsign-compare -Wundef
>>         -Wpointer-arith -Wvolatile-register-var -Wformat
>>         -Wformat-security -Wduplicated-branches -Wduplicated-cond],
>>         [CFLAGS])
>>         AX_APPEND_COMPILE_FLAGS([-Wall -Wextra -Wsign-compare -Wundef
>>         -Wpointer-arith -Wvolatile-register-var -Wformat
>>         -Wformat-security -Wduplicated-branches -Wduplicated-cond],
>>         [CXXFLAGS])
>> 
>>         -AX_APPEND_COMPILE_FLAGS([-Wbad-function-cast -Wwrite-strings
>>         -Wmissing-prototypes -Werror-implicit-function-declaration
>>         -Winit-self -Wlogical-op -Wshadow=local
>>         -Wmultistatement-macros], [CFLAGS])
>>         +AX_APPEND_COMPILE_FLAGS([-Wbad-function-cast -Wwrite-strings
>>         -Wmissing-prototypes -Werror-implicit-function-declaration
>>         -Winit-self -Wlogical-op -Wshadow=local
>-Wmultistatement-macros
>>         -Werror=incompatible-pointer-types], [CFLAGS])
>>         AX_APPEND_COMPILE_FLAGS([-pipe], [CFLAGS])
>> 
>>         dnl
>> 
>> 
>>         -- 
>>         Envoyé de mon appareil Android avec Courriel K-9 Mail.
>Veuillez
>>         excuser
>>         ma brièveté.
>>        
>------------------------------------------------------------------------
>>         vlc-devel mailing list
>>         To unsubscribe or modify your subscription options:
>>         https://mailman.videolan.org/listinfo/vlc-devel
>> 
>>    
>------------------------------------------------------------------------
>>     vlc-devel mailing list
>>     To unsubscribe or modify your subscription options:
>>     https://mailman.videolan.org/listinfo/vlc-devel
>> 
>> 
>> -- 
>> Envoyé de mon appareil Android avec Courriel K-9 Mail. Veuillez
>excuser 
>> ma brièveté.
>> 
>> _______________________________________________
>> vlc-devel mailing list
>> To unsubscribe or modify your subscription options:
>> https://mailman.videolan.org/listinfo/vlc-devel
>> 
>_______________________________________________
>vlc-devel mailing list
>To unsubscribe or modify your subscription options:
>https://mailman.videolan.org/listinfo/vlc-devel

-- 
Envoyé de mon appareil Android avec Courriel K-9 Mail. Veuillez excuser ma brièveté.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mailman.videolan.org/pipermail/vlc-devel/attachments/20191002/dffb60d0/attachment.html>