[Spice-devel] [PATCH v2 09/13] Add mention of fall-through comments

Christophe de Dinechin christophe.de.dinechin at gmail.com
Wed Feb 7 14:53:06 UTC 2018



> On 7 Feb 2018, at 14:58, Frediano Ziglio <fziglio at redhat.com> wrote:
> 
>> 
>> From: Christophe de Dinechin <dinechin at redhat.com>
>> 
>> Signed-off-by: Christophe de Dinechin <dinechin at redhat.com>
>> ---
>> docs/spice_style.txt | 2 ++
>> 1 file changed, 2 insertions(+)
>> 
>> diff --git a/docs/spice_style.txt b/docs/spice_style.txt
>> index ccc60bb3..e35272ec 100644
>> --- a/docs/spice_style.txt
>> +++ b/docs/spice_style.txt
>> @@ -293,6 +293,8 @@ default:
>> }
>> ----
>> 
>> +Use /* Fall through */ comments when there is no break (compilers will emit
>> a warning otherwise)
>> +
>> Types indentation
>> ~~~~~~~~~~~~~~~~~
>> 
> 
> This comment has not much to do with the "Branching indentation" section
> beside is under a switch example

Any other place where you would put it?

> 
> Frediano
> _______________________________________________
> Spice-devel mailing list
> Spice-devel at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/spice-devel



More information about the Spice-devel mailing list