[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.
1034-135 at online.de
Mon Apr 23 23:48:55 CEST 2012
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). AFAIK avi files do not
support setting an AR, so why should the fmt.video not be initialized
with 1 right from the start or where would I find the correct position
to initialize the correct AR? It seemed to work (at least up to now...)
Thanks for the hint, though.
More information about the vlc-devel