diff options
author | Domagoj Trsan <domagoj.trsan@gmail.com> | 2014-09-20 12:40:38 (GMT) |
---|---|---|
committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2014-09-24 06:41:25 (GMT) |
commit | d1d1a9d3a104531e25b7a33d60b627e93700327f (patch) | |
tree | f4ae1adf06506f0d6dc2e2470a8e5c1c90201b84 /drivers/staging/line6 | |
parent | c74920124750e5270c689d8c892a1c4263e5a547 (diff) | |
download | linux-d1d1a9d3a104531e25b7a33d60b627e93700327f.tar.xz |
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 <domagoj.trsan@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Diffstat (limited to 'drivers/staging/line6')
-rw-r--r-- | drivers/staging/line6/midibuf.c | 11 |
1 files changed, 8 insertions, 3 deletions
diff --git a/drivers/staging/line6/midibuf.c b/drivers/staging/line6/midibuf.c index f0adb7b..1ff8569 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) |