[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