Re: [PATCH 2/3] winecoreaudio: Implement the lock-free callback design.

[Peter Rosin]

[e-mail] skrev 2012-01-30 18:30:
> Of the same vein, one liners like
> EnterCS
> This-> playing = StateStopped;
> LeaveCS
> are suspicious too.  What does that guarantee?

Perhaps that another thread doing as below isn't fooled into
calling both one() and three()?

EnterCS
if (This-> playing == StateRunning) {
  one();
}
two();
if (This-> playing == StateStopped) {
  three();
}
LeaveCS

I don't know whether, or not, that has any bearing what-so-ever on
the patch under discussion...

Cheers,
Peter