summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorWei Yang2019-03-04 08:16:31 +0100
committerLaurent Vivier2019-05-02 18:12:58 +0200
commit25d68ffb6bca333ce8496eca7f438d5a93f9cd06 (patch)
tree97c03335af2621be169b12a1011548be77f03c0b
parentCODING_STYLE: specify the indent rule for multiline code (diff)
downloadqemu-25d68ffb6bca333ce8496eca7f438d5a93f9cd06.tar.gz
qemu-25d68ffb6bca333ce8496eca7f438d5a93f9cd06.tar.xz
qemu-25d68ffb6bca333ce8496eca7f438d5a93f9cd06.zip
CODING_STYLE: indent example code as all others
All the example code are indented with four spaces except this one. Fix this by adding four spaces here. Signed-off-by: Wei Yang <richardw.yang@linux.intel.com> Reviewed-by: Eric Blake <eblake@redhat.com> Reviewed-by: Philippe Mathieu-Daudé <philmd@redhat.com> Reviewed-by: Igor Mammedov <imammedo@redhat.com> Reviewed-by: Stefano Garzarella <sgarzare@redhat.com> Message-Id: <20190304071631.27567-3-richardw.yang@linux.intel.com> Signed-off-by: Laurent Vivier <laurent@vivier.eu>
-rw-r--r--CODING_STYLE8
1 files changed, 4 insertions, 4 deletions
diff --git a/CODING_STYLE b/CODING_STYLE
index 90321e9c28..cb8edcbb36 100644
--- a/CODING_STYLE
+++ b/CODING_STYLE
@@ -147,10 +147,10 @@ block to a separate function altogether.
When comparing a variable for (in)equality with a constant, list the
constant on the right, as in:
-if (a == 1) {
- /* Reads like: "If a equals 1" */
- do_something();
-}
+ if (a == 1) {
+ /* Reads like: "If a equals 1" */
+ do_something();
+ }
Rationale: Yoda conditions (as in 'if (1 == a)') are awkward to read.
Besides, good compilers already warn users when '==' is mis-typed as '=',