From: Christophe de Dinechin <>

As written, the headers style guide looks quite wrong. In particular,
it places headers in an order that makes it hard to detect hidden
dependencies in SPICE headers.

These rules can be enforced by the .clang-format proposed in earlier patch,
locally if you use the Emacs clang-format.el integration supplied with LLVM.

Signed-off-by: Christophe de Dinechin <>
 docs/spice_style.txt | 44 +++++++++++++++++++++++++++++---------------
 1 file changed, 29 insertions(+), 15 deletions(-)

diff --git a/docs/spice_style.txt b/docs/spice_style.txt
index 108a57a5..ff505b2a 100644
--- a/docs/spice_style.txt
+++ b/docs/spice_style.txt
@@ -407,34 +407,48 @@ Historically, some headers added underscores liberally, 
e.g. MY_MODULE_H_. This
 Header inclusion
-Headers should be included in this order
+Headers should be included in this order:
+- config.h, which should only be included from C source files
+- [module].h, where [module].c is the corresponding implementation file
+- [module]-xyz.h, which are support headers for [module]
+- Other application headers, using #include "file.h"
+- System headers, using #include <file.h>
+- If necessary, C++ system headers, using #include <file>
+This order is designed to maximize chances of catching missing headers in 
headers (i.e. headers that are not self-contained).
+In summary, Headers should be included in this order
-#include <system_headers.h>
-#include <no_spice_no_system_libraries.h>
+#include "config.h"
+#include "source.h"
+#include "source-support.h"
+#include "some-other-source.h"
 #include <spice_protocol.h>
 #include <spice_common.h>
-#include "spice_server.h"
+#include <no_spice_no_system_libraries.h>
+#include <system_headers.h>
+#include <vector>
+#include <cstdio>
-(note the empty line between no spice-server and spice-server headers)
+(note the empty line between application headers included with "" and system 
headers included with <>
-Also in source (no header) files you must include `config.h` at the beginning 
so should start (beside comments and copyright) with
+Headers should include only the headers required to process the header itself, 
and otherwise include as little as possible.
-#include <config.h>
+#ifndef SOURCE_H
+#define SOURCE_H
+#include "application-header-required-for-header.h"
-#include <system_headers.h>
-#include <no_spice_no_system_libraries.h>
-#include <spice_protocol.h>
-#include <spice_common.h>
+#include <system-header-required-for-header.h>
-#include "spice_server.h"
+#endif /* SOURCE_H */
2.13.5 (Apple Git-94)

Spice-devel mailing list

Reply via email to