On Thu, Sep 12, 2013 at 08:17:15PM +0200, Luca Barbato wrote:
> Initial cparser support.
> ---

The second line is redundant, drop it.

> --- a/configure
> +++ b/configure
> @@ -3980,6 +3999,9 @@ elif enabled clang; then
>      check_cflags -Werror=implicit-function-declaration
>      check_cflags -Werror=missing-prototypes
>      check_cflags -Werror=return-type
> +elif enabled cparser; then
> +    check_cflags -Wno-missing-variable-declarations
> +    check_cflags -Wno-empty-statement
>  elif enabled armcc; then
>      # 2523: use of inline assembler is deprecated
>      add_cflags -W${armcc_opt},--diag_suppress=2523

Why not add_cflags?  Is there a chance cparser might not support
these flags?

Diego
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to