[vlc-devel] [PATCH 2/2] According to the AVI-specs. the pixels are square. Therefore the aspect ratio should be set. Otherwise the aspect ratio of a previous video might be used.

[Laurent Aimar]

On Mon, Apr 23, 2012 at 11:48:55PM +0200, VlcVelope wrote:
> On 22.04.2012 22:04, Laurent Aimar wrote:
>> On Sun, Apr 22, 2012 at 04:40:05PM +0200, VlcVelope wrote:
>>> ---
>>>   modules/demux/avi/avi.c |    1 +
>>>   1 files changed, 1 insertions(+), 0 deletions(-)
>>>
>>> diff --git a/modules/demux/avi/avi.c b/modules/demux/avi/avi.c
>>> index 08659d9..25a4d9c 100644
>>> --- a/modules/demux/avi/avi.c
>>> +++ b/modules/demux/avi/avi.c
>>> @@ -530,6 +530,7 @@ static int Open( vlc_object_t * p_this )
>>>                   fmt.video.i_bits_per_pixel = p_vids->p_bih->biBitCount;
>>>                   fmt.video.i_frame_rate = tk->i_rate;
>>>                   fmt.video.i_frame_rate_base = tk->i_scale;
>>> +                fmt.video.i_sar_num = fmt.video.i_sar_den = 1;
>>   Usually the AR in AVI comes from the ES stream. If you force the AR at
>> the demuxer level, it won't work anymore.
>>   Also, the AR doesn't go across streams (at least it's the case in 2.0
>> and master).
>>
>> Regards,
>>
> You are probably right. But still why do some avi-files play with an  
> incorrect AR (they do not with WMP and others).

Can you upload a file that show that issue (but does play right with WMP)?

Regards,

-- 
fenrir