[FFmpeg-devel] [PATCH 2/3] configure: check and use -Werror=empty-body

James Darnley james.darnley at gmail.com
Fri Apr 4 00:53:55 CEST 2014


On 2014-04-04 00:45, James Darnley wrote:
> Would have caught the trailing semi-colon in lavu/log.c that broke detection of
> 256 colour capable terminals.
> ---
>  configure |    1 +
>  1 files changed, 1 insertions(+), 0 deletions(-)
> 
> diff --git a/configure b/configure
> index 46452ee..cb76890 100755
> --- a/configure
> +++ b/configure
> @@ -4776,6 +4776,7 @@ check_cflags -Wundef
>  check_cflags -Wmissing-prototypes
>  check_cflags -Wno-pointer-to-int-cast
>  check_cflags -Wstrict-prototypes
> +check_cflags -Werror=empty-body
>  enabled extra_warnings && check_cflags -Winline
>  
>  check_disable_warning(){
> 

Is it ever not an error?

This will make fate turn a violent shade of red at present.  I would
suggest waiting to hear back from Tomas, or anyone else, about the mxf
change before applying this.

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 683 bytes
Desc: OpenPGP digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20140404/7f4872a9/attachment.asc>


More information about the ffmpeg-devel mailing list