[MPlayer-dev-eng] [PATCH] fix mencoder creates bad avi files when recording h264 in ts container from DVB

Nico Sabbi Nicola.Sabbi at poste.it
Thu Jul 2 15:06:35 CEST 2009


On Thursday 02 July 2009 14:17:06 Attila Kinali wrote:
> On Thu, 02 Jul 2009 16:11:21 +0400
>
> vladisslav vladisslav <vladisslav at inbox.ru> wrote:
> > > > > +   sh_video->disp_w=picture.display_picture_width;
> > > > > +   sh_video->disp_h=picture.display_picture_height;
> > > >
> > > > Any comments on this patch?
> > > > It seems to be trival.
> > >
> > > Well, if you are confident and tested it thoroughly, apply..
> >
> > I have used mplayer/mencoder (various svn builds) with this patch
> > applied since January 2009 and got no problem. So i think it does
> > not break anything.
>
> The question isnt whether it breaks anything, but whether the patch
> is correct. Although it might not break anything now, it might be
> still wrong and cause problems later when something at a very
> different point is changed.
>

picture must be  memset(0) before calling parse_sps(), as it should 
have been from the beginning, my fault. After that the patch is 
correct





More information about the MPlayer-dev-eng mailing list