<div dir="ltr"><div>Hi,</div><div>Turns out a more portable way is to replace /* fallthrough */ with /* fall through */.</div><div>This should work will older versions of GCC and Clang as well.</div><div>I'll send a different patch.<br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Fri, Mar 22, 2019 at 1:11 PM Awais Chishti <<a href="mailto:chishtiawais511@gmail.com">chishtiawais511@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr">I'll look into it right now. Thanks.<br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Fri, Mar 22, 2019 at 1:09 PM Steve Lhomme <<a href="mailto:robux4@ycbcr.xyz" target="_blank">robux4@ycbcr.xyz</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Hi,<br>
<br>
 From a quick look it seems that this attribute was added in gcc 7 but <br>
we support older versions of gcc. So you need to check the gcc version.<br>
<br>
Also you may need to check if it works with Clang as well and since what <br>
version.<br>
<br>
On 3/21/2019 8:10 PM, <a href="mailto:chishtiawais511@gmail.com" target="_blank">chishtiawais511@gmail.com</a> wrote:<br>
> From: Awais Chishti <<a href="mailto:chishtiawais511@gmail.com" target="_blank">chishtiawais511@gmail.com</a>><br>
><br>
> ---<br>
>   include/vlc_common.h | 13 +++++++++++++<br>
>   1 file changed, 13 insertions(+)<br>
><br>
> diff --git a/include/vlc_common.h b/include/vlc_common.h<br>
> index 033c618677..f3eb10271a 100644<br>
> --- a/include/vlc_common.h<br>
> +++ b/include/vlc_common.h<br>
> @@ -185,6 +185,19 @@<br>
>   # define VLC_USED<br>
>   #endif<br>
>   <br>
> +/**<br>
> + * Explicitly indicate fallthrough<br>
> + *<br>
> + * Use this macro as a replacement for //fallthrough comments in switch blocks.<br>
> + *<br>
> + * This may disable inaccurate compiler warnings of implicit fallthrough.<br>
> + */<br>
> +#ifdef __GNUC__<br>
> +# define VLC_FALLTHROUGH __attribute__ ((fallthrough));<br>
> +#else<br>
> +# define VLC_FALLTHROUGH<br>
> +#endif<br>
> +<br>
>   #if defined (__ELF__) || defined (__MACH__)<br>
>   # define VLC_WEAK __attribute__((weak))<br>
>   #else<br>
> -- <br>
> 2.21.0<br>
><br>
> _______________________________________________<br>
> vlc-devel mailing list<br>
> To unsubscribe or modify your subscription options:<br>
> <a href="https://mailman.videolan.org/listinfo/vlc-devel" rel="noreferrer" target="_blank">https://mailman.videolan.org/listinfo/vlc-devel</a><br>
<br>
_______________________________________________<br>
vlc-devel mailing list<br>
To unsubscribe or modify your subscription options:<br>
<a href="https://mailman.videolan.org/listinfo/vlc-devel" rel="noreferrer" target="_blank">https://mailman.videolan.org/listinfo/vlc-devel</a></blockquote></div>
</blockquote></div>