> On 16 Feb 2018, at 10:05, Christophe Fergeau <cfergeau@xxxxxxxxxx> wrote: > > On Thu, Feb 15, 2018 at 05:06:19PM +0100, Christophe de Dinechin wrote: >> From: Christophe de Dinechin <dinechin@xxxxxxxxxx> >> >> Signed-off-by: Christophe de Dinechin <dinechin@xxxxxxxxxx> >> --- >> 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. Agreed, done. > > Christophe _______________________________________________ Spice-devel mailing list Spice-devel@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/spice-devel