On 10/8/2017 9:02 PM, Mark Thompson wrote: > On 09/10/17 00:34, Carl Eugen Hoyos wrote: >> 2017-10-09 1:26 GMT+02:00 Mark Thompson <s...@jkqxz.net>: >>> On 08/10/17 22:08, Carl Eugen Hoyos wrote: >>>> ffmpeg | branch: master | Carl Eugen Hoyos <ceffm...@gmail.com> | Sun Oct >>>> 8 23:08:09 2017 +0200| [c2d155e11ee5ec732d471982f2dee43703bcd5a7] | >>>> committer: Carl Eugen Hoyos >>>> >>>> configure: Disable -Wbool-operation. >>>> >>>> Requested-by: Ronald and Derek >>>> >>>>> http://git.videolan.org/gitweb.cgi/ffmpeg.git/?a=commit;h=c2d155e11ee5ec732d471982f2dee43703bcd5a7 >>>> --- >>>> >>>> configure | 1 + >>>> 1 file changed, 1 insertion(+) >>>> >>>> diff --git a/configure b/configure >>>> index 77c9a18c3c..d5e96e95e1 100755 >>>> --- a/configure >>>> +++ b/configure >>>> @@ -6291,6 +6291,7 @@ check_cflags -Wmissing-prototypes >>>> check_cflags -Wno-pointer-to-int-cast >>>> check_cflags -Wstrict-prototypes >>>> check_cflags -Wempty-body >>>> +check_cflags -Wno-bool-operation >>>> >>>> if enabled extra_warnings; then >>>> check_cflags -Wcast-qual >>>> >>> >>> Following this I am now getting a lot of: >>> >>> src/libavformat/dump.c: At top level: >>> cc1: warning: unrecognized command line option ‘-Wno-bool-operation’ >> >> Please revert, sorry for the mess! > > Ok, reverted for now. > > I do agree with the intent of the patch, though, so we should fix this. It > looks like it will affect all future warnings with GCC, so it probably needs > a new function in configure to do the warning test in a different way somehow. > > Thanks, > > - Mark
This test should probably use check_disable_warning() instead of check_cflags(). _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
