[Openal-devel] Trunk + 0.0.8 (was: Yet another source queue race condition)

[Adam D. Moss]

Elias Naur wrote:
> P.S. Please do not hesitate to comment or propose a correct fix on my earlier 
> reported queue race condition too.

By this point I think you know more about the code in
question than *anyone* except for jv, but this looks like
good debugging to me (& I've had no problems so far running
with the previous queue race condition fix we discussed).

I don't know whether a source lock or a context lock would
be the right thing to grab for this latter problem, though
the source lock sounds more likely.

I'm quite inclined to get the likely-looking fixes commited
pretty much now[1] (unless jv objects) and let them bake for
a few weeks before restarting the 0.0.8 release process
with CVS HEAD, including the trunk work there plus D. Hageman's
spec and openalrc fixes, because of the glut of fixes that
have come in during the 0.0.8-pre cycle.

I'm happy enough that the earlier race fix doesn't cause
more bogosity than it fixes, and when this latter fix boils
down to acquiring a lock I think that sounds at least harmless
enough to let into the trunk for a few weeks (please submit a
patch if you can).

--Adam
-- 
Adam D. Moss   -   adam at gimp.org