[MPlayer-dev-eng] Re: [PATCH] VFCTRL_PERIODIC_UPDATE

[Rich Felker]

On Fri, Jan 19, 2007 at 09:36:34PM +0200, Uoti Urpala wrote:
> On Fri, 2007-01-19 at 19:14 +0100, Reimar Döffinger wrote:
> > On Fri, Jan 19, 2007 at 07:17:05PM +0200, Uoti Urpala wrote:
> > > 1) Remove the forced unpause when handling slave commands.
> > > 
> > > 2) Change timing sleep to do a select on input fds instead of plain
> > > nanosleep/usleep and process any input immediately (at least on Unix,
> > > doing this on Windows would require more work).
> > > 
> > > If I've understood correctly what you intend to do with the overlay
> > > filter then after those changes implementing it should be a lot more
> > > straightforward. It could use ordinary slave commands which would remove
> > > the need for both the input filter hacks mentioned earlier and
> > > VFCTRL_PERIODIC_UPDATE (since the only mentioned use for the update was
> > > basically to check input instead of any real filter functionality).
> > 
> > Huh? To check input yes, but to update the picture on screen, too. How
> > to do that without either something like VFCTRL_PERIODIC_UPDATE or
> > unpausing (which is not okay).
> 
> The slave command can invoke whatever code VFCTRL_PERIODIC_UPDATE would
> end up executing (which in this case seems to be a filter call and
> vo->flip_page()).

Not necessarily possible. The image might no longer exist in any
readable memory at the time this occurs. :( Try looking at vf_menu. I
think it keeps a backup copy of the last picture to make this
possible..

Rich