thanks, so we can use e.g. '__attribute__((fallthrough))' statement;
looks ok? (modified patch attached)
2018-02-22 11:10 GMT+01:00 David Woodhouse <dwmw2 at infradead.org>:
> On Thu, 2018-02-22 at 00:12 +0100, ?ubom?r Carik wrote:
>> Signed-off-by: ?ubom?r Carik <Lubomir.Carik at gmail.com>
>> ---
>> main.c | 2 ++
>> 1 file changed, 2 insertions(+)
>>
>> diff --git a/main.c b/main.c
>> index 935147f..86ba8cb 100644
>> --- a/main.c
>> +++ b/main.c
>> @@ -610,6 +610,7 @@ static void print_build_opts(void)
>> case 2:
>> printf("%sHOTP software token", sep);
>> sep = comma;
>> + break;
>> case 1:
>> printf("%sTOTP software token", sep);
>> sep = comma;
>
> That one really should fall through. If it returns 1 that means we have
> only TOTP support; if it returns 2 we have both.
>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-Solve-few-fall-trough-warnings.patch
Type: application/octet-stream
Size: 917 bytes
Desc: not available
URL: <http://lists.infradead.org/pipermail/openconnect-devel/attachments/20180222/d383792d/attachment.obj>
