Re: clang 10 -Wimplicit-fallthrough

[Date Prev] [Date Next] [Thread Prev] [Thread Next] [Date Index] [Thread Index]

 



Darren Tucker wrote:
> Annotating these points with a FALLTHROUGH macro would make more
> work keeping the code in sync and so is currently a non-starter.

Upstream OpenSSH/OpenBSD has no interest in embracing such a macro?

Isn't there some clang/llvm intent in OpenBSD?


> diff --git a/aclocal.m4 b/aclocal.m4
> index 25ecc49a..fca940dd 100644
> --- a/aclocal.m4
> +++ b/aclocal.m4
> @@ -21,6 +21,11 @@ int main(int argc, char **argv) {
>  	double m = l / 0.5;
>  	long long int n = argc * 12345LL, o = 12345LL * (long long int)argc;
>  	printf("%d %d %d %f %f %lld %lld\n", i, j, k, l, m, n, o);
> +	switch(i){
> +	case 0: j += i;
> +		/* FALLTHROUGH */
> +	default: j += k;
> +	}

Are you thinking to also add a test case for when it's missing?


//Peter
_______________________________________________
openssh-unix-dev mailing list
openssh-unix-dev@xxxxxxxxxxx
https://lists.mindrot.org/mailman/listinfo/openssh-unix-dev



[Date Prev] [Date Next] [Thread Prev] [Thread Next] [Date Index] [Thread Index]

[Index of Archives]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux]     [Linux OMAP]     [Linux MIPS]     [ECOS]     [Asterisk Internet PBX]     [Linux API]

  Powered by Linux