diff options
| author |   Takashi Iwai <tiwai@suse.de> | 2005-10-10 11:46:31 +0200 |
|---|---|---|
| committer |   Jaroslav Kysela <perex@suse.cz> | 2005-11-04 13:17:42 +0100 |
| commit | 7c22f1aaa23370bf9ba2dd3abbccbed70dced216 (patch) | |
| tree | ebc9c1e5cfdab4815afdfab1ba583e6220287252 /sound/core/pcm_native.c | |
| parent | [ALSA] Sasha Khapyorsky: My email address is changed, there is update. (diff) | |
| download | linux-dev-7c22f1aaa23370bf9ba2dd3abbccbed70dced216.tar.xz linux-dev-7c22f1aaa23370bf9ba2dd3abbccbed70dced216.zip | |
[ALSA] Remove snd_runtime_check() macro
Remove snd_runtime_check() macro.
This macro worsens the readability of codes. They should be either
normal if() or removable asserts.
Also, the assert displays stack-dump, instead of only the last caller
pointer.
Signed-off-by: Takashi Iwai <tiwai@suse.de>
Diffstat (limited to 'sound/core/pcm_native.c')
| -rw-r--r-- | sound/core/pcm_native.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/sound/core/pcm_native.c b/sound/core/pcm_native.c index e97b2d162cc7..e6e2b70314c0 100644 --- a/sound/core/pcm_native.c +++ b/sound/core/pcm_native.c @@ -2053,7 +2053,8 @@ static int snd_pcm_open(struct inode *inode, struct file *file) snd_pcm_file_t *pcm_file; wait_queue_t wait; - snd_runtime_check(device >= SNDRV_MINOR_PCM_PLAYBACK && device < SNDRV_MINOR_DEVICES, return -ENXIO); + if (device < SNDRV_MINOR_PCM_PLAYBACK || device >= SNDRV_MINOR_DEVICES) + return -ENXIO; pcm = snd_pcm_devices[(cardnum * SNDRV_PCM_DEVICES) + (device % SNDRV_MINOR_PCMS)]; if (pcm == NULL) { err = -ENODEV; |