[Spice-devel] [PATCH v2 08/13] Remove confusing example

Christophe de Dinechin christophe at dinechin.org
Wed Feb 7 11:07:20 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 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)



More information about the Spice-devel mailing list