[Spice-devel] [PATCH v3 07/11] Remove confusing example

Christophe de Dinechin christophe at dinechin.org
Thu Feb 8 11:25:27 UTC 2018


From: Christophe de Dinechin <dinechin at redhat.com>

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 at redhat.com>
---
 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 486efbc9..68dbeeef 100644
--- a/docs/spice_style.txt
+++ b/docs/spice_style.txt
@@ -279,15 +279,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)



More information about the Spice-devel mailing list