[Spice-devel] [PATCH v3 00/11] Updates to the style guide

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


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

Changes in v3:

- Integrated clarification regarding headers
- Added external references as a starting point
- Changed [source,h] annotations (unsure if that's accepted)
- Changed brace placement from Allman to Linux in .clang-format

Changes in v2:

- This splits the earlier patch about the style guide into its individual
  components. This also clarifies the intent of some of the changes in the
  commit messages.

- Fixed typos

Christophe de Dinechin (11):
  Add .clang-format with defaults matching what's specified in the style
    guide
  Added External References section
  Specify file extensions for C++ and Obj-C
  Rephrase assertion section
  Rephrase section about constants
  Rephrase section on short functions for readability
  Remove confusing example
  Add mention of fall-through comments
  Add mention of header guards
  Add guidelines about warnings and whitespaces
  Rewrite the style guide for headers

 .clang-format        |  23 +++++++++
 docs/spice_style.txt | 139 ++++++++++++++++++++++++++++++++++++++++-----------
 2 files changed, 132 insertions(+), 30 deletions(-)
 create mode 100644 .clang-format

-- 
2.13.5 (Apple Git-94)



More information about the Spice-devel mailing list