Re: [Gnash-dev] First stab at fixing an OpenGL bug

[Quinn Storm]

On Sunday 03 June 2007 13:12:59 strk wrote:
> On Sun, Jun 03, 2007 at 01:02:27PM -0400, Quinn Storm wrote:
> > On Sunday 03 June 2007 04:10:03 strk wrote:
> > > Ok, last proposal: just call ::display again, so the stage is rendered
> > > again on expose events. The only possible downside I can think of is
> > > expose events showing some intermediate states otherwise not visible
> > > with normal playback (consider an sub-FPS interval transforming
> > > characters), which doesn't seem a big problem to me.
> > >
> > >
> > > --strk;
> >
> > That sounds good, and would have been what I'd have done had I known
> > about ::display properly.  I'll look into that, but it sounds like a
> > proper solution at least for the moment.
>
> We discussed in the past about adding a ::redraw method to the Gui class,
> to be also used from a menu item.  Can't remember if we did already, if we
> didn't consider adding one yourself (you'd call that on expose).
>
> Also, to avoid redrawing as hell while receiving a long chain of expose
> events it might be worth having a loop checking for it. IIRC Udo added one
> mechanism working at SWF FPS rate (wantsRedraw or similar method) but we
> might want an SWF agnostic one. See also the Movie_Advancement page on
> gnashdev wiki.
>
> Have fun !
>
> --strk;


Here we go, I hooked in using just the want_redraw (with an ugly #ifdef since 
the function isn't yet exported to glue), I haven't pushed the change also to 
sdl yet, but this patch fixes gtk.  If you like it, I'm sure its easy to push 
the change to SDL

opengl_fix_redraw.diff
Description: Text Data