[MPlayer-dev-eng] [PATCH] Prevent ao_win32 from hanging during uninit(0)

Reimar Döffinger Reimar.Doeffinger at stud.uni-karlsruhe.de
Thu Mar 5 22:17:46 CET 2009


On Thu, Mar 05, 2009 at 03:59:52PM -0500, David Bolen wrote:
> Reimar Döffinger <Reimar.Doeffinger at stud.uni-karlsruhe.de> writes:
> > On Thu, Mar 05, 2009 at 02:47:59PM -0500, David Bolen wrote:
> >> Any downside to just bounding the loop for now?  I do have a
> >> self-interest in looking into this further but in the meantime, it's a
> >> hang exposure which the patch corrects.  Certainly it's an
> >> improvement, even if imperfect, over the existing code.
> >
> > Well, I am undecided if it is worth the effort, I consider just going
> > with the ao_sdl solution (which in some ways is also more correct),
> > namely:
> >>  if (!immed)
> >>    usec_sleep(get_delay() * 1000 * 1000);
> 
> Yep, you win, that's cleaner :-)

I'll probably apply that one somewhen soon then.

> Was your "worth the effort" point in regards to any patch at all or
> just my particular one?  I still think preventing the possibility of
> an infinite loop in uninit() to be worth it, even if it doesn't solve
> all the problems with the module.

I expressed myself badly, I meant I was wondering if the code complexity
your patch added was reasonable for the functionality, particularly
compared to my suggestion above. It seems to me you consider your
approach not to be preferable to my suggestion (I think my sugggestion
could sometimes result in truncated sound - this could probably not
happen with the current code).



More information about the MPlayer-dev-eng mailing list