[FFmpeg-devel] DV Video and SAR

[Baptiste Coudurier]

Hi,

On 10/14/09 12:58 PM, Charles Yates wrote:
> Hi all,
>
> Wondered if you might consider the attached patches to dv handling.
>
> First part corrects the SAR for anamorphic DVCPRO - basically, 960x720 16:9
> material has a SAR of 4:3 (giving a presentation of 1280x720).

Thanks, applied.

> Second part enables the determination of the SAR from a direct use of the
> codec (as opposed to restricting it to only being available via the avformat
> raw dv avformat parser). There seems to be a hint in ffplay that SAR
> reporting at the decode level should migrate to the AVFrame (would be nice
> :-)), so this is certainly moving it in the right direction - but it's a
> minimalist effort on my part, effectively copy/pasting the relevant parts
> from the avformat side. Probably better to refactor correctly in parallel
> with the AVFrame migration, but figured that it's better than nothing for
> now.

This one looks ok as well. I'll apply it soon I think.

I was wondering if we shouldn't set sar for dv to digital sar as opposed 
to analog sar considering mpeg-2 sar and how FFmpeg deals with sar in 
general. Any opinion ?

-- 
Baptiste COUDURIER
Key fingerprint                 8D77134D20CC9220201FC5DB0AC9325C5C1ABAAA
FFmpeg maintainer                                  http://www.ffmpeg.org