[v4,07/12] Remove confusing example

Submitted by Christophe de Dinechin on Feb. 15, 2018, 4:06 p.m.


Message ID 20180215160625.75406-8-christophe@dinechin.org
State New
Headers show
Series "Updates to the style guide" ( rev: 3 ) in Spice

Not browsing as part of any series.

Commit Message

Christophe de Dinechin Feb. 15, 2018, 4:06 p.m.
From: Christophe de Dinechin <dinechin@redhat.com>

The sentence explaining that example makes no real sense,
the mentioned style is not used by the current code,
and the coding style suggestion is horrendous.

Not to mention that it flies in the face of all automatic indentation
tools I know of.

Change since v3:
- Make wording of commit log "less kind", used Frediano's suggestion

Signed-off-by: Christophe de Dinechin <dinechin@redhat.com>
 docs/spice_style.txt | 10 +---------
 1 file changed, 1 insertion(+), 9 deletions(-)

Patch hide | download patch | download mbox

diff --git a/docs/spice_style.txt b/docs/spice_style.txt
index 9e6536e4..977ce154 100644
--- a/docs/spice_style.txt
+++ b/docs/spice_style.txt
@@ -319,15 +319,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.
-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