[Mplayer-cvslog] CVS: main/libmpdemux demux_mkv.cpp,1.42,1.43

[Felix Buenemann]

On Sunday 26 October 2003 22:15, Attila Kinali wrote:
> On Sun, 26 Oct 2003 22:09:57 +0100
>
> Attila Kinali CVS <attila at mplayerhq.hu> wrote:
> > * avoid multiple segfaults when there are more then SUB_MAX_TEXT-1 lines
> > at the same time
> >
> > +  if (mkv_d->subs.lines> SUB_MAX_TEXT - 1 )
> > +  {
> > +  	mp_msg(MSGT_DEMUX, MSGL_WARN,"[mkv] Warning: too many sublines to
> > render, skipping\n"); +  	mkv_d->subs.lines--;
> > +  	return;
> > +  }
>
> Moritz, can you go trough demux_mkv.cpp and fix this ? The segfault comes
> from the clearing while loop at line 2867
>
> Also, has anyone a good idea what to do if some ****** encoder thinks
> he has to put every word of a sentence on its own line ?
> At the moment this code just drops them after the 9th word.
How about caching em up until a reasonable line length is reached and then 
flusing it out to the osd?

> 			Attila Kinali

-- 
Best Regards,
        Atmos
____________________________________________
- MPlayer Developer - http://mplayerhq.hu/ -
____________________________________________