[FFmpeg-devel] [PATCH] Clarifying the use of the scale filter interlace option

[Tim Nicholson]

On 02/04/12 19:41, Mark Himsley wrote:
> On 02/04/2012 08:56, Tim Nicholson wrote:
>> On 31/03/12 12:54, Mark Himsley wrote:
>>> On 29/03/2012 20:33, Michael Niedermayer wrote:
>>>> On Thu, Mar 29, 2012 at 08:49:38AM +0100, Tim Nicholson wrote:
>>>
>>>>>
>>>>> Also unlike other cases -1 (auto) is not the default.
>>>>
>>>> this could be changed if it works better overall
>>>
>>> I made this point on 25/07/2011, with a patch. It was rejected because
>>> the progressive videos used in fate tests were always flagged as
>>> progressive in DV codecs so PSNR tests failed horribly.
>>
>> Did you mean "flagged as interlaced". Progressive marked as progressive
>> is surely correct.
> 
> No. I meant what I said :-)

Sorry I am still confused....

> DV is always flagged as interlaced by dvvideo_decode_frame()


But earlier you said "..were always flagged as
>>> progressive in DV codecs..."

How can both be true?

> 
>     s->picture.interlaced_frame = 1;
>     s->picture.top_field_first  = 0;
> 


-- 
Tim