OSDN Git Service

pcm: minor code cleanup for ioctl call
authorTakashi Sakamoto <o-takashi@sakamocchi.jp>
Fri, 23 Jun 2017 13:09:24 +0000 (22:09 +0900)
committerTakashi Iwai <tiwai@suse.de>
Fri, 23 Jun 2017 13:33:50 +0000 (15:33 +0200)
When error occurs, return value from ioctl(2) is -1 and error code can
be got thread local variable, errno. It's OK just to check the return
value without any assignment.

Signed-off-by: Takashi Sakamoto <o-takashi@sakamocchi.jp>
Signed-off-by: Takashi Iwai <tiwai@suse.de>
src/pcm/pcm_hw.c

index f0efcde..de6fd1a 100644 (file)
@@ -132,8 +132,7 @@ static int sync_ptr1(snd_pcm_hw_t *hw, unsigned int flags)
 {
        int err;
        hw->sync_ptr->flags = flags;
-       err = ioctl((hw)->fd, SNDRV_PCM_IOCTL_SYNC_PTR, (hw)->sync_ptr);
-       if (err < 0) {
+       if (ioctl(hw->fd, SNDRV_PCM_IOCTL_SYNC_PTR, hw->sync_ptr) < 0) {
                err = -errno;
                SYSMSG("SNDRV_PCM_IOCTL_SYNC_PTR failed (%i)", err);
                return err;