summaryrefslogtreecommitdiffstats
path: root/eng/coding-80cols.rst
blob: 8fbe10dbc4aecf7ac224b33a6397b1c721fac2f1 (plain) (blame)
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
.. comment SPDX-License-Identifier: CC-BY-SA-4.0

.. COMMENT: COPYRIGHT (c) 2018.
.. COMMENT: RTEMS Foundation, The RTEMS Documentation Project

Eighty Character Line Limit
***************************

.. COMMENT: TBD - Convert the following to Rest and insert into this file
.. COMMENT: TBD - https://devel.rtems.org/wiki/Developer/Coding/80_characters_per_line

 Code should look good for everyone under some standard width assumptions.
 Where a line wraps should be the same for anyone reading the code. For
 historical reasons, RTEMS uses 80 characters as the maximum width for each
 line of code.

If you find yourself with code longer than 80 characters, first ask yourself
whether the nesting level is too deep, names too long, compound expressions too
complicated, or if some other guideline for improving readability can help to
shrink the line length. Refactoring nested blocks into functions can help to
alleviate code width problems while improving code readability. Making names
descriptive yet terse can also improve readability. If absolutely necessary
to have a long line, follow the rules on this page to break the line up to adhere
to the 80 characters per line rule.

Breaking long lines
-------------------

``if``, ``while``, and ``for`` loops have their condition expressions aligned
and broken on separate lines. When the conditions have to be broken, none go on
the first line with the ``if``, ``while``, or ``for`` statement, and none go on
the last line with the closing parenthesis and (optional) curly brace. Long
statements are broken up and indented at operators, with an operator always
being the last token on a line. No blank spaces should be left at the end of
any line. Here is an example with a ``for`` loop.

.. code-block:: c

  for ( initialization = statement; a + really + long + statement + that + evaluates + to < a + boolean; another + statement++ ) {
    z = a + really + long + statement + that + needs + two + lines + gets + indented + four + more + spaces + on + the + second + and + subsequent + lines + and + broken + up + at + operators;
  }

Should be replaced with

.. code-block:: c

  for (
    initialization = statement;
    a + really + long + statement + that + evaluates + to <
    a + boolean;
    another + statement++
  ) {
    z = a + really + long + statement + that + needs +
        two + lines + gets + indented + four + more +
        spaces + on + the + second + and + subsequent +
        lines + and + broken + up + at + operators;
  }

Note that indentations should add 2 nesting levels (4 space characters, not tabs).

Similarly,

.. code-block:: c

  if ( this + that < those && this + these < that && this + those < these && this < those && those < that ) {

should be broken up like

.. code-block:: c

  if (
    this + that < those &&
    this + these < that &&
    this + those < these &&
    this < those &&
    those < that
  ) {

Note that each expression that resolves to a boolean goes on its own line.
Where you place the boolean operator is a matter of choice.

When a line is long because of a comment at the end, move the comment to
just before the line, for example

.. code-block:: c

  #define A_LONG_MACRO_NAME (AND + EXPANSION) /* Plus + a + really + long + comment */

can be replaced with

.. code-block:: c

  /* Plus + a + really + long + comment */
  #define A_LONG_MACRO_NAME (AND + EXPANSION)

C Preprocessor macros need to be broken up with some care, because the
preprocessor does not understand that it should eat newline characters. So

.. code-block:: c

  #define A_LONG_MACRO_NAME (AND + EXCESSIVELY + LONG + EXPANSION + WITH + LOTS + OF + EXTRA + STUFF + DEFINED)

would become

.. code-block:: c

  #define A_LONG_MACRO_NAME ( \
    AND + EXCESSIVELY + LONG + EXPANSION + WITH + LOTS + OF + EXTRA + STUFF + \
    DEFINED \
  )

Notice that each line is terminated by a backslash then the carriage return.
The backslash tells the preprocessor to eat the newline. Of course, if you have
such a long macro, you should consider not using a macro.

Function declarations can be broken up at each argument, for example

.. code-block:: c

  int this_is_a_function( int arg1, int arg2, int arg3, int arg4, int arg5, int arg6, int arg7, int arg8, int arg9 );

would be broken up as

.. code-block:: c

  int this_is_a_function(
    int arg1,
    int arg2,
    int arg3,
    int arg4,
    int arg5,
    int arg6,
    int arg7,
    int arg8,
    int arg9
  );

Excessively long comments should be broken up at a word boundary or somewhere
that makes sense, for example

.. code-block:: c

  /* Excessively long comments should be broken up at a word boundary or somewhere that makes sense, for example */

would be

.. code-block:: c

  /* Excessively long comments should be broken up at a word boundary or
   * somewhere that makes sense, for example */

Note that multiline comments have a single asterisk aligned with the asterisk
in the opening ``/*``. The closing ``*/`` should go at the end of the last
line.