[MPlayer-dev-eng] [PATCH] ASS subtitle speedup (take 2)

Reimar Döffinger Reimar.Doeffinger at stud.uni-karlsruhe.de
Mon Feb 5 19:18:06 CET 2007


Hello,
On Mon, Dec 04, 2006 at 12:23:16AM +0300, Evgeniy Stepanov wrote:
> On Saturday 02 December 2006 23:22, Reimar Döffinger wrote:
> > On Sat, Dec 02, 2006 at 10:59:31PM +0300, Evgeniy Stepanov wrote:
> > > On Saturday 02 December 2006 22:33, Reimar Döffinger wrote:
> > > > > > Though I wonder if/how the imgs->imgs == NULL && imgs->changed == 1
> > > > > > case should be handled (your code probably would crash?).
> > > > >
> > > > > Why should it ? imgs->imgs == NULL is nothing special, just a list of
> > > > > zero elements.
> > > >
> > > > Because the original patch did
> > > > goto skip_upload;
> > > > without checking img!=NULL and the code at skip_upload did assume it to
> > > > be != NULL IIRC.
> > >
> > > I still don't see how it depends on that, but ok, extra check won't hurt.
> > > A new version fixing everything that was discussed.
> >
> > No obvious problems (also had a quick look at the other code).
> 
> In fact there is one: subtitles going from "disabled" to "enabled" (either 
> with 'j' or with track switching) does not count as a change, and display is 
> not updated. My solution is inelegant but seems to work. Any ideas ?

This or something similar has been applied btw.
Try to say when you apply stuff, makes it easier to keep my mailbox
clean *g*

Greetings,
Reimar Döffinger



More information about the MPlayer-dev-eng mailing list