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

Reply via email to