From: Christophe de Dinechin <dinechin@xxxxxxxxxx> The sentence explaining that example makes no real sense, and the coding style suggestion is horrendous (not to mention flies in the face of all automatic indentation tools) Signed-off-by: Christophe de Dinechin <dinechin@xxxxxxxxxx> --- docs/spice_style.txt | 10 +--------- 1 file changed, 1 insertion(+), 9 deletions(-) diff --git a/docs/spice_style.txt b/docs/spice_style.txt index a9d77afa..ccc60bb3 100644 --- a/docs/spice_style.txt +++ b/docs/spice_style.txt @@ -254,15 +254,7 @@ if (condition) { + In case of long condition statement, prefer having additional temporary variable over multiple line condition statement. + -In case of new line in condition statement. -+ -[source,c] ----- -if (long_name && very_long_name && very_long || - var_name) { ----- -+ -or indent under the round bracket using spaces +Indent long conditionals under the opening parenthesis using spaces + [source,c] ---- -- 2.13.5 (Apple Git-94) _______________________________________________ Spice-devel mailing list Spice-devel@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/spice-devel