On 14/03/17 07:23, Luca Barbato wrote:
> On 14/03/2017 02:38, Mark Thompson wrote:
>> +#define wloop(mag, wr) \
>> + for (n = 0; n < n_in; n++) { \
>> + wr; \
>> + ptr += mag + 1; \
>> + }
>
> Either wrap-in/use a do-while() or drop the ;
Sure, done.
> I'd rather have it using bitstream from beginning.
I would prefer not to make nontrivial changes to working code written by
someone else.
> Beside that seems fine.
On 14/03/17 14:20, Diego Biurrun wrote:
> On Tue, Mar 14, 2017 at 01:38:00AM +0000, Mark Thompson wrote:
>> --- /dev/null
>> +++ b/libavcodec/vp9_superframe_bsf.c
>> @@ -0,0 +1,206 @@
>> +/*
>> + * Vp9 invisible (alt-ref) frame to superframe merge bitstream filter
>
> nit: VP9
Fixed.
Thanks,
- Mark
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel