On 19/09/13 19:14, Diego Biurrun wrote:
> 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?
Can be done and will be pushed with add_cflags.
lu
_______________________________________________
libav-devel mailing list
libav-devel@libav.org
https://lists.libav.org/mailman/listinfo/libav-devel
