[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.
fenrir at elivagar.org
Tue Apr 24 00:20:38 CEST 2012
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).
> 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)?
More information about the vlc-devel