[MPlayer-cvslog] r22295 - trunk/libmpcodecs/vd_ffmpeg.c

[Michael Niedermayer]

Hi

On Mon, Feb 26, 2007 at 12:09:16AM +0300, Evgeniy Stepanov wrote:
> On Tuesday 20 February 2007 23:09, nicodvb wrote:
> > Author: nicodvb
> > Date: Tue Feb 20 21:09:29 2007
> > New Revision: 22295
> >
> > Modified:
> >    trunk/libmpcodecs/vd_ffmpeg.c
> >
> > Log:
> > reinit the vo when aspect ratio changes; patch by C.E. Hoyos approved by
> > Michael
> 
> With this change, aspect ration is ignored in anamorphic mkv. The attached 
> patch fixes it, but I'm not sure it is the right way.

well i dont think it is correct, strictly speaking files which break with the
current aspect code are broken but that doesnt solve the problem ...

so maybe checking if sh->aspect is 0 in the first call, and depending on that
either always setting it or never touching it might solve things
or checking if sh->aspect is 0 || matches last_sample_aspect_ratio

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

I do not agree with what you have to say, but I'll defend to the death your
right to say it. -- Voltaire
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/mplayer-cvslog/attachments/20070226/d8fc4ad4/attachment.pgp>