On 2/11/2014 7:37 PM, Diego Biurrun wrote:
>> +#include "libavutil/internal.h"
>> +#include "libavutil/common.h"
>> +#include "libavutil/opt.h"
>> +#include "libavutil/pixdesc.h"
>> +#include "avcodec.h"
>> +#include "internal.h"
>> +
>> +#include <x265.h>
> 
> Reorder and move the system header before the local ones.

Done locally. 

>> +    memset(&x265pic_out, 0, sizeof(x265pic_out));
> 
> x265_picture x265pic_out = { 0 };

Causes additional compiler warnings with gcc and clang:

/home/daemon404/dev/l/libav/libavcodec/libx265.c:173:34: warning: suggest 
braces around initialization of subobject [-Wmissing-braces]
    x265_picture x265pic_out = { 0 };
                                 ^
                                 {}

> Move .long_name after .name.

Done locally.

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

Reply via email to