netdev
[Top] [All Lists]

Re: [PATCH] Missing break in switch statement.

To: davem@xxxxxxxxxx, davej@xxxxxxxxxx
Subject: Re: [PATCH] Missing break in switch statement.
From: YOSHIFUJI Hideaki <yoshfuji@xxxxxxxxxxxxxxxx>
Date: Tue, 12 Aug 2003 02:14:14 +0900 (JST)
Cc: netdev@xxxxxxxxxxx, yoshfuji@xxxxxxxxxxxxxx
In-reply-to: <E19mFqr-00068T-00@tetrachloride>
References: <E19mFqr-00068T-00@tetrachloride>
Sender: netdev-bounce@xxxxxxxxxxx
In article <E19mFqr-00068T-00@tetrachloride> (at Mon, 11 Aug 2003 17:48:57 
+0100), davej@xxxxxxxxxx says:

> Is this intentional? It should at least have a 
> /* FALLTHROUGH */ or similar if so.

Of course not.

>               else
>                       val = opt->offset;
> +             break;
>  
>       default:
>               return -ENOPROTOOPT;

agreed.

--yoshfuji

<Prev in Thread] Current Thread [Next in Thread>