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.
>
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
