From: Domagoj Trsan Date: Sat, 20 Sep 2014 12:40:38 +0000 (+0200) Subject: staging: line6: fix midibuf.c coding style issue X-Git-Url: http://drtracing.org/?a=commitdiff_plain;h=d1d1a9d3a104531e25b7a33d60b627e93700327f;p=deliverable%2Flinux.git staging: line6: fix midibuf.c coding style issue Fix the following checkpatch.pl warning: - else is not generally useful after a break or return Signed-off-by: Domagoj Trsan Signed-off-by: Greg Kroah-Hartman --- diff --git a/drivers/staging/line6/midibuf.c b/drivers/staging/line6/midibuf.c index f0adb7baa603..1ff856989fd6 100644 --- a/drivers/staging/line6/midibuf.c +++ b/drivers/staging/line6/midibuf.c @@ -15,11 +15,14 @@ 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)