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

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



From: Christophe de Dinechin <dinechin@xxxxxxxxxx>

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)

_______________________________________________
Spice-devel mailing list
Spice-devel@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/spice-devel




[Index of Archives]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux]     [Linux OMAP]     [Linux MIPS]     [ECOS]     [Asterisk Internet PBX]     [Linux API]     [Monitors]