[FFmpeg-devel] [Patch] New filter -- dejudder

Nicholas Robbins nickrobbins at yahoo.com
Wed Jan 29 23:20:44 CET 2014



> On Wednesday, January 29, 2014 5:06 PM, Lukasz Marek <lukasz.m.luki at gmail.com> wrote:

> >>>>    +typedef struct {
>>>>    +    const AVClass *class;
>>>>    +    int64_t *ringbuff;
>>>>    +    int a,b,c,d;
>>> 
>>>  Can they be more meaningful?
>> 
>>  They are indices to a circular buffer. I wanted them to be short. If it is 
> preferable I could change them to two_previous, previous, ultimate, penultimate. 
> It will make the "new_pts +=..." line in filter_frame longer.
> 
> OK. I don't mind, but a,b,c,d looks awful.
I couldn't think of something short but meaningful. I'm open to suggestions. 
>> 

> It seems you attached old patch :P
> 
> 
> -- 
> Best Regards,
> Lukasz Marek

As I said rookie mistakes :). New patch in response to Andrey Utkin's email.

--
Nicholas Robbins


More information about the ffmpeg-devel mailing list