mirrors/alsa-lib
1
0
Fork 0
mirror of https://github.com/alsa-project/alsa-lib.git synced 2025-11-03 09:01:52 -05:00
Code Issues Projects Releases Packages Wiki Activity Actions 1

pcm: Remove assert() from snd_pcm_hw_params_slave()

Browse source 
Using assert() for non-fatal error checks is really brain-dead.
These are especially bad, as it hits even in the normal operation with
plugins and some h/w constraints.

Reported-by: Alan Horstmann <gineera@aspect135.co.uk>
Signed-off-by: Takashi Iwai <tiwai@suse.de>
This commit is contained in:
Takashi Iwai 2015-05-21 07:26:39 +02:00
parent b4222f3fdc
commit 67f73b0fab
1 changed files with 4 additions and 2 deletions
Download patch file Download diff file Expand all files Collapse all files

6
src/pcm/pcm_params.c
View file

@ -2244,9 +2244,11 @@ int snd_pcm_hw_params_slave(snd_pcm_t *pcm, snd_pcm_hw_params_t *params,
snd_pcm_hw_params_t slave_params; snd_pcm_hw_params_t slave_params;
int err; int err;
err = sprepare(pcm, &slave_params); err = sprepare(pcm, &slave_params);
assert(err >= 0); if (err < 0)
return err;
err = schange(pcm, params, &slave_params); err = schange(pcm, params, &slave_params);
assert(err >= 0); if (err < 0)
return err;
err = sparams(pcm, &slave_params); err = sparams(pcm, &slave_params);
if (err < 0) if (err < 0)
cchange(pcm, params, &slave_params); cchange(pcm, params, &slave_params);