[MPlayer-dev-eng] [PATCH] Allow aspect change in TS and DVB

[Ivan Kalvachev]

2006/4/25, Nico Sabbi <nicola_sabbi at fastwebnet.it>:
> Nico Sabbi wrote:
>
> > Carl Eugen Hoyos wrote:
> >
> >>
> >> Index: libmpcodecs/vd.c
> >> ===================================================================
> >> RCS file: /cvsroot/mplayer/main/libmpcodecs/vd.c,v
> >> retrieving revision 1.84
> >> diff -u -r1.84 vd.c
> >> --- libmpcodecs/vd.c    24 Mar 2006 08:12:02 -0000      1.84
> >> +++ libmpcodecs/vd.c    25 Apr 2006 01:45:26 -0000
> >> @@ -289,6 +289,8 @@
> >>     }
> >>     if(sh->aspect>0.01){
> >>       int w;
> >> +      if (sh->ds->demuxer->file_format == DEMUXER_TYPE_MPEG_TS &&
> >> sh->format ==
> >> 0x10000002)
> >> +        fill_aspect_info(&sh->aspect);
> >>       mp_msg(MSGT_CPLAYER,MSGL_INFO,MSGTR_MovieAspectIsSet,sh->aspect);
> >>       w=(int)((float)screen_size_y*sh->aspect); w+=w%2; // round
> >>       // we don't like horizontal downscale || user forced width:
> >>
> >>
> >>
> >>
> >
> > there's no need to check the demuxer type here
> >
>
> and, btw the same applies to 0x1000001 (mpeg1) and to all other
> ids/fourccs assigned to mpeg1 and mpeg2.
> For a list see libmpdemux/muxer_mpeg.c function is_mpeg12().
>
> btw2, this patch should also work for ffmpeg12. Does it?

The right place to fix this bug is in vd_mpeg12.c or libmpeg2.

ffmpeg12 already detects and re-initializes the video system on aspect change.