On Thu, Feb 15, 2018 at 05:06:19PM +0100, Christophe de Dinechin wrote:
> From: Christophe de Dinechin <dinec...@redhat.com>
> 
> Signed-off-by: Christophe de Dinechin <dinec...@redhat.com>
> ---
>  docs/spice_style.txt | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/docs/spice_style.txt b/docs/spice_style.txt
> index 996c5cd9..9e6536e4 100644
> --- a/docs/spice_style.txt
> +++ b/docs/spice_style.txt
> @@ -178,7 +178,7 @@ If multiple related constants are to be defined, consider 
> the use of enumeration
>  Short functions
>  ---------------
>  
> -Try to split code to short functions, each having simple functionality, in 
> order to improve code readability and re-usability. Prefix with inline short 
> functions or functions that were splitted for readability reason.
> +Try to split code to short functions, each having simple functionality, in 
> order to improve code readability and re-usability. Prefix with `inline` 
> functions that were splitted for readability reason or that are very short.

I'd just drop the second sentence. The compiler usually knows better.

Christophe

Attachment: signature.asc
Description: PGP signature

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

Reply via email to