diff options
author | FeRD (Frank Dana) <ferdnyc@gmail.com> | 2021-08-09 09:18:55 +0000 |
---|---|---|
committer | FeRD (Frank Dana) <ferdnyc@gmail.com> | 2021-08-09 09:18:55 +0000 |
commit | 0984b872b1094a2af5fd6c17ce20e873b7e7793e (patch) | |
tree | 4d4f1aa8f1d6b02eecd72e65acd8be5dace47669 /CODING_STYLE.md | |
parent | ea3676aceeecb1f99c0969eb368ebe3cb6a5f206 (diff) | |
download | pango-0984b872b1094a2af5fd6c17ce20e873b7e7793e.tar.gz |
CODING_STYLE: Fence code samples, fix tab breakage
Diffstat (limited to 'CODING_STYLE.md')
-rw-r--r-- | CODING_STYLE.md | 52 |
1 files changed, 35 insertions, 17 deletions
diff --git a/CODING_STYLE.md b/CODING_STYLE.md index aa183e5d..d9456439 100644 --- a/CODING_STYLE.md +++ b/CODING_STYLE.md @@ -9,29 +9,34 @@ In brief overview: get a separate indentation level, so the total indent for an enclosed block is 4 characters. + + ```c if (x < foo (y, z)) haha = bar[4] + 5; else { - while (z) - { - haha += foo (z, z); - z--; - } - return abc (haha); + while (z) + { + haha += foo (z, z); + z--; + } + return abc (haha); } + ``` - Spaces should be present between function name and argument block, and after commas. - foo (z, z) + foo (z, z) - In pointer types, the '*' is grouped with the variable name, not with the base type. - int *a; + int *a; + + Not: - NOT: 'int* a'. + int* a; In cases where there is no variable name, for instance, return values, there should be a single space between the base type @@ -47,6 +52,7 @@ Documentation comments All public API functions should have inline documentation headers in the gtk-doc / gnome-doc style. For instance: +```c /** * pango_layout_get_line: * @layout: a `PangoLayout` @@ -66,34 +72,44 @@ PangoLayoutLine * pango_layout_get_line (PangoLayout *layout, int line) [...] - +``` Choosing Function Names ======================= - Don't abbreviate in unexpected ways: - pango_layout_get_line_count () + ```c + pango_layout_get_line_count (); + ``` Not: - pango_layout_ln_cnt () + ```c + pango_layout_ln_cnt (); + ``` - function that retrieve a value in a side-effect free fashion, should include "get" in the name. - int pango_layout_get_line_count (PangoLayout *layout) + ```c + int pango_layout_get_line_count (PangoLayout *layout); + ``` - not + Not: - pango_layout_line_count () + ```c + pango_layout_line_count (); + ``` - functions that set a single parameter in a side-effect free fashion should include "set" in the name, for instance: + ```c void pango_layout_set_width (PangoLayout *layout, - int width); + int width); + ``` Other comments ============== @@ -103,7 +119,9 @@ Other comments If width is unsigned and 10, then: - int new_width = MAX (width - 15, 1); + ```c + int new_width = MAX (width - 15, 1); + ``` produces 4294967291, not 1. |