[FFmpeg-devel] [PATCH] reitnerlace - tinterlace-like filter under LGPL

Vasile Toncu vasile.toncu at tremend.com
Tue Mar 13 17:10:20 EET 2018



On 06.03.2018 20:38, Thomas Mundt wrote:
> Hi,
>
> 2018-03-05 13:48 GMT+01:00 Carl Eugen Hoyos <ceffmpeg at gmail.com>:
>
>> 2018-03-05 12:37 GMT+01:00, Paul B Mahol <onemda at gmail.com>:
>>> On 3/5/18, Vasile Toncu <vasile.toncu at tremend.com> wrote:
>>>> Hello,
>>>>
>>>> Thanks for the review. I've made changes according to your guidance.
>>>>
>>>> It would be great to know if the community will go on with our intention
>>>> of adding reinterlace as a alternative for tinterlace.
>>>>
>>>> That being said, here is the new patch.
>>> As already said, this is not acceptable.
>>>
>>> There is no point in having 2 filters with near same funcionality.
>> If you consider the new filter ok, the existing filter will be removed
>> in the same push. I believe sending only the new filter makes
>> reviewing easier.
>>
> For me reviewing would be easier when Vasile sends a patchset that includes
> the replacement of tinterlace filter.
> That way existing fate tests could be used which are fortunately pretty
> extensive in this case.
> Also it would be helpful when you and/or other experienced ffmpeg
> developers would clarify first which parts of tinterlace have to be
> rewritten for proper relicensing.
> Being left in the dark makes working on patches frustrating.
>
> Another question is how to deal with vf_interlace? IMHO for the user there
> should be no difference in output, speed and license.
> Two options:
> 1. Relicensing and slice threading will also be ported to vf_interlace
> 2. The commands from vf_interlace will be included in the new tinterlace
> filter. vf_interlace will be deleted together with old tinterlace filter
>
> I would prefer the second option, but maybe there are even better options
> that donĀ“t come to my mind.
>
> Please comment.
> Thanks,
> Thomas
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel at ffmpeg.org
> http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Hello everyone,

sorry for a delayed response.

 From what has been discussed in here, I think the reinterlace will 
exist with tinterlace for a period of time, just after that the 
tinterlace can be removed.

To have the reinterlace added, what is needed to be done from my side?

Thanks,
Vasile Toncu

Thanks


More information about the ffmpeg-devel mailing list