ALSA: echoaudio: remove redundant assignment to variable bytes
authorColin Ian King <colin.i.king@gmail.com>
Wed, 2 Mar 2022 17:07:28 +0000 (17:07 +0000)
committerTakashi Iwai <tiwai@suse.de>
Fri, 4 Mar 2022 08:50:39 +0000 (09:50 +0100)
The variable bytes is being assigned a value that is never read, it
is being re-assigned inside a following if block. The assignment is
redundant and can be removed.

Cleans up clang scan build warning:
sound/pci/echoaudio/midi.c:211:9: warning: Although the value stored
to 'bytes' is used in the enclosing expression, the value is never
actually read from 'bytes' [deadcode.DeadStores]

Signed-off-by: Colin Ian King <colin.i.king@gmail.com>
Link: https://lore.kernel.org/r/20220302170728.1094633-1-colin.i.king@gmail.com
Signed-off-by: Takashi Iwai <tiwai@suse.de>
sound/pci/echoaudio/midi.c

index cb72d27..7be5c33 100644 (file)
@@ -208,7 +208,7 @@ static void snd_echo_midi_output_write(struct timer_list *t)
 
        /* No interrupts are involved: we have to check at regular intervals
        if the card's output buffer has room for new data. */
-       sent = bytes = 0;
+       sent = 0;
        spin_lock_irqsave(&chip->lock, flags);
        chip->midi_full = 0;
        if (!snd_rawmidi_transmit_empty(chip->midi_out)) {