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

Reimar Döffinger Reimar.Doeffinger at stud.uni-karlsruhe.de
Sat Dec 2 21:22:35 CET 2006


Hello,
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).

> +  ass_image_t *img;
>    ass_image_t *i;
>    int cnt;
> +
> +  img = imgs->imgs;

Move the initialization up to the declaration before committing (no need
to send a new patch because of this of course).

Greetings,
Reimar Döffinger



More information about the MPlayer-dev-eng mailing list