Commit 985df9cd authored by Maarten Lankhorst's avatar Maarten Lankhorst Committed by Alexandre Julliard

winealsa: Remove forceformat and 2 orphan LeaveCriticalSection.

parent 5eabdff6
...@@ -73,7 +73,6 @@ struct IDsDriverImpl ...@@ -73,7 +73,6 @@ struct IDsDriverImpl
/* IDsDriverImpl fields */ /* IDsDriverImpl fields */
IDsDriverBufferImpl* primary; IDsDriverBufferImpl* primary;
UINT wDevID; UINT wDevID;
DWORD forceformat;
}; };
struct IDsDriverBufferImpl struct IDsDriverBufferImpl
...@@ -396,8 +395,6 @@ static HRESULT SetFormat(IDsDriverBufferImpl *This, LPWAVEFORMATEX pwfx) ...@@ -396,8 +395,6 @@ static HRESULT SetFormat(IDsDriverBufferImpl *This, LPWAVEFORMATEX pwfx)
{ {
if (errno != EBUSY || !This->pcm) if (errno != EBUSY || !This->pcm)
{ {
/* **** */
LeaveCriticalSection(&This->pcm_crst);
WARN("Cannot open sound device: %s\n", snd_strerror(err)); WARN("Cannot open sound device: %s\n", snd_strerror(err));
return DSERR_GENERIC; return DSERR_GENERIC;
} }
...@@ -407,8 +404,6 @@ static HRESULT SetFormat(IDsDriverBufferImpl *This, LPWAVEFORMATEX pwfx) ...@@ -407,8 +404,6 @@ static HRESULT SetFormat(IDsDriverBufferImpl *This, LPWAVEFORMATEX pwfx)
err = snd_pcm_open(&pcm, WOutDev[This->drv->wDevID].pcmname, SND_PCM_STREAM_PLAYBACK, SND_PCM_NONBLOCK); err = snd_pcm_open(&pcm, WOutDev[This->drv->wDevID].pcmname, SND_PCM_STREAM_PLAYBACK, SND_PCM_NONBLOCK);
if (err < 0) if (err < 0)
{ {
/* **** */
LeaveCriticalSection(&This->pcm_crst);
WARN("Cannot open sound device: %s\n", snd_strerror(err)); WARN("Cannot open sound device: %s\n", snd_strerror(err));
return DSERR_BUFFERLOST; return DSERR_BUFFERLOST;
} }
...@@ -433,12 +428,7 @@ static HRESULT SetFormat(IDsDriverBufferImpl *This, LPWAVEFORMATEX pwfx) ...@@ -433,12 +428,7 @@ static HRESULT SetFormat(IDsDriverBufferImpl *This, LPWAVEFORMATEX pwfx)
err = snd_pcm_hw_params_set_rate_near(pcm, hw_params, &rate, NULL); err = snd_pcm_hw_params_set_rate_near(pcm, hw_params, &rate, NULL);
if (err < 0) { rate = pwfx->nSamplesPerSec; WARN("Could not set rate\n"); goto err; } if (err < 0) { rate = pwfx->nSamplesPerSec; WARN("Could not set rate\n"); goto err; }
if (!ALSA_NearMatch(rate, pwfx->nSamplesPerSec) && (This->drv->forceformat++)) if (!ALSA_NearMatch(rate, pwfx->nSamplesPerSec))
{
WARN("Could not set exact rate %d, instead %d, bombing out\n", pwfx->nSamplesPerSec, rate);
goto err;
}
else if (!ALSA_NearMatch(rate, pwfx->nSamplesPerSec))
{ {
WARN("Could not set sound rate to %d, but instead to %d\n", pwfx->nSamplesPerSec, rate); WARN("Could not set sound rate to %d, but instead to %d\n", pwfx->nSamplesPerSec, rate);
pwfx->nSamplesPerSec = rate; pwfx->nSamplesPerSec = rate;
...@@ -496,7 +486,6 @@ static HRESULT WINAPI IDsDriverBufferImpl_SetFormat(PIDSDRIVERBUFFER iface, LPWA ...@@ -496,7 +486,6 @@ static HRESULT WINAPI IDsDriverBufferImpl_SetFormat(PIDSDRIVERBUFFER iface, LPWA
/* **** */ /* **** */
EnterCriticalSection(&This->pcm_crst); EnterCriticalSection(&This->pcm_crst);
This->drv->forceformat = FALSE;
hr = SetFormat(This, pwfx); hr = SetFormat(This, pwfx);
/* **** */ /* **** */
LeaveCriticalSection(&This->pcm_crst); LeaveCriticalSection(&This->pcm_crst);
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment