[Spice-devel] [PATCH v4 00/12] Updates to the style guide

Christophe de Dinechin christophe at dinechin.org
Thu Feb 15 16:06:13 UTC 2018


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

Changes in v4:
- Integrate the many comments made on list
  Details in the respective commit logs

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 (12):
  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 guidelines about warnings
  Fix the style guide for headers
  Integrate suggestions from Christophe Fergeau
  Point out that the coding style is for all SPICE components
  Whitespace adjustment guideline

 .clang-format        |  30 +++++++++
 docs/spice_style.txt | 171 +++++++++++++++++++++++++++++++++++++++++----------
 2 files changed, 168 insertions(+), 33 deletions(-)
 create mode 100644 .clang-format

-- 
2.13.5 (Apple Git-94)



More information about the Spice-devel mailing list