staging: line6: fix midibuf.c coding style issue
authorDomagoj Trsan <domagoj.trsan@gmail.com>
Sat, 20 Sep 2014 12:40:38 +0000 (14:40 +0200)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Wed, 24 Sep 2014 06:41:25 +0000 (23:41 -0700)
Fix the following checkpatch.pl warning:
- else is not generally useful after a break or return

Signed-off-by: Domagoj Trsan <domagoj.trsan@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/line6/midibuf.c

index f0adb7b..1ff8569 100644 (file)
 
 static int midibuf_message_length(unsigned char code)
 {
+       int message_length;
+
        if (code < 0x80)
-               return -1;
+               message_length = -1;
        else if (code < 0xf0) {
                static const int length[] = { 3, 3, 3, 3, 2, 2, 3 };
-               return length[(code >> 4) - 8];
+
+               message_length = length[(code >> 4) - 8];
        } else {
                /*
                   Note that according to the MIDI specification 0xf2 is
@@ -29,8 +32,10 @@ static int midibuf_message_length(unsigned char code)
                static const int length[] = { -1, 2, -1, 2, -1, -1, 1, 1, 1, 1,
                        1, 1, 1, -1, 1, 1
                };
-               return length[code & 0x0f];
+               message_length = length[code & 0x0f];
        }
+
+       return message_length;
 }
 
 static int midibuf_is_empty(struct midi_buffer *this)