ALSA: emu8000: Use swap macro in snd_emu8000_sample_new
authorGustavo A. R. Silva <gustavo@embeddedor.com>
Tue, 17 Jul 2018 15:06:10 +0000 (10:06 -0500)
committerTakashi Iwai <tiwai@suse.de>
Tue, 17 Jul 2018 15:18:03 +0000 (17:18 +0200)
Make use of the swap macro and remove unnecessary variable *tmp*. This
makes the code easier to read and maintain.

This code was detected with the help of Coccinelle.

Signed-off-by: Gustavo A. R. Silva <gustavo@embeddedor.com>
Signed-off-by: Takashi Iwai <tiwai@suse.de>
sound/isa/sb/emu8000_patch.c

index c2e41d2..d45a6b9 100644 (file)
@@ -165,11 +165,8 @@ snd_emu8000_sample_new(struct snd_emux *rec, struct snd_sf_sample *sp,
                return 0;
 
        /* be sure loop points start < end */
-       if (sp->v.loopstart > sp->v.loopend) {
-               int tmp = sp->v.loopstart;
-               sp->v.loopstart = sp->v.loopend;
-               sp->v.loopend = tmp;
-       }
+       if (sp->v.loopstart > sp->v.loopend)
+               swap(sp->v.loopstart, sp->v.loopend);
 
        /* compute true data size to be loaded */
        truesize = sp->v.size;