ALSA: emu10k1_patch: Use swap macro in snd_emu10k1_sample_new
authorGustavo A. R. Silva <gustavo@embeddedor.com>
Tue, 17 Jul 2018 15:00:43 +0000 (10:00 -0500)
committerTakashi Iwai <tiwai@suse.de>
Tue, 17 Jul 2018 15:17:52 +0000 (17:17 +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/pci/emu10k1/emu10k1_patch.c

index 0e069ae..c32eb70 100644 (file)
@@ -70,11 +70,8 @@ snd_emu10k1_sample_new(struct snd_emux *rec, struct snd_sf_sample *sp,
                loopend = sampleend;
 
        /* 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 + BLANK_HEAD_SIZE;