[FFmpeg-devel] [PATCH] Print pixel and screen aspect ratios

[Michel Bardiaux]

Emanuele Fumagalli a ?crit :
> Michael Niedermayer a ?crit :
>> Hi
>>
>> On Thu, Oct 18, 2007 at 03:07:01PM +0200, Michel Bardiaux wrote:
>>> Robert Swain a ?crit :
>>>> Hello,
>>>> On 18 Oct 2007, at 13:48, Michel Bardiaux wrote:
>>>>> Valid point. DAR it is. New patch attached.
>>>> Could you use display_aspect_ratio rather than screen_aspect_ratio as
>>>> well? It doesn't really matter but for consistency. :)
>>> "Consistency is the hobgoblin of small minds", but since The Lord of The
>>> Rings and Harry Potter, goblins are k00l, so yes 8-|) New patch.
>> patch ok
> 
> I tested on a mp4 file I have with PAR 16:9 and it reports PAR 0:1.
> Could it be it needs to decode a video packet before?
> I know that because on the player I built I have to check the 
> sample_aspect_ratio just after the first decode.

Probably right. The patch reports whatever info was set in 
sample_aspect_ratio by the codec at the time avcodec_string is called.

-- 
Michel Bardiaux
R&D Director
T +32 [0] 2 790 29 41
F +32 [0] 2 790 29 02
E mailto:mbardiaux at mediaxim.be

Mediaxim NV/SA
Vorstlaan 191 Boulevard du Souverain
Brussel 1160 Bruxelles
http://www.mediaxim.com/