[MPlayer-dev-eng] [PATCH] ASS - detect changes to speed up -vo gl

Evgeniy Stepanov eugeni.stepanov at gmail.com
Sun Oct 29 15:28:53 CET 2006


On Sunday 29 October 2006 17:09, Reimar Döffinger wrote:
> Hello,
>
> On Sun, Oct 29, 2006 at 05:43:21PM +0400, Evgeniy Stepanov wrote:
> > On Saturday 28 October 2006 20:05, Jindrich Makovicka wrote:
> > > the attached patch adds a subtitle change detection to libass. It is a
> > > quite brute force approach, but it significantly reduces the cpu load
> > > when using -vo gl, as the texture uploads are costly (at least on
> > > nVidia 6600 + build 9625). With the patch, the textures are uploaded
> > > only when the EOSD actually changes.
> >
> > I suggest removing images_root and making it caller's responsibility to
> > deallocate the list of images. Libass will provide functions for
> > comparing two lists and for deallocating a list. Then, vo_gl can keep
> > ass_images_prev and compare it with the new one.
>
> And who will be responsible for deallocating the images themselves? I
> think this way will complicate things a lot.

In current implementation they are never deallocated.

In future, a reference counter can be added to bitmap_t. This will require 
changing ass_image_t a bit (a pointer to bitmap_t should be included in it).



More information about the MPlayer-dev-eng mailing list