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 <dw...@infradead.org>:
> On Thu, 2018-02-22 at 00:12 +0100, Ľubomír Carik wrote:
>> Signed-off-by: Ľubomír Carik <lubomir.ca...@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.
>

Attachment: 0001-Solve-few-fall-trough-warnings.patch
Description: Binary data

_______________________________________________
openconnect-devel mailing list
openconnect-devel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/openconnect-devel

Reply via email to