CodingStyle.txt
author Florian Pose <fp@igh-essen.com>
Fri, 28 Oct 2011 11:31:54 +0200
branchstable-1.5
changeset 2125 af3eda069078
child 2127 0814802f5928
permissions -rw-r--r--
Added coding style document.
2125
af3eda069078 Added coding style document.
Florian Pose <fp@igh-essen.com>
parents:
diff changeset
     1
$Id$
af3eda069078 Added coding style document.
Florian Pose <fp@igh-essen.com>
parents:
diff changeset
     2
af3eda069078 Added coding style document.
Florian Pose <fp@igh-essen.com>
parents:
diff changeset
     3
vim: spelllang=en tw=78
af3eda069078 Added coding style document.
Florian Pose <fp@igh-essen.com>
parents:
diff changeset
     4
af3eda069078 Added coding style document.
Florian Pose <fp@igh-essen.com>
parents:
diff changeset
     5
This is a short introduction to the coding style that shall be used. The below
af3eda069078 Added coding style document.
Florian Pose <fp@igh-essen.com>
parents:
diff changeset
     6
rules are applicable for all C source files, except the Ethernet drivers, for
af3eda069078 Added coding style document.
Florian Pose <fp@igh-essen.com>
parents:
diff changeset
     7
which the Linux Kernel coding style shall be used to minimize the
af3eda069078 Added coding style document.
Florian Pose <fp@igh-essen.com>
parents:
diff changeset
     8
differences).
af3eda069078 Added coding style document.
Florian Pose <fp@igh-essen.com>
parents:
diff changeset
     9
af3eda069078 Added coding style document.
Florian Pose <fp@igh-essen.com>
parents:
diff changeset
    10
* Lines shall not exceed 78 characters.
af3eda069078 Added coding style document.
Florian Pose <fp@igh-essen.com>
parents:
diff changeset
    11
af3eda069078 Added coding style document.
Florian Pose <fp@igh-essen.com>
parents:
diff changeset
    12
* Indentation shall be done using 4 space characters
af3eda069078 Added coding style document.
Florian Pose <fp@igh-essen.com>
parents:
diff changeset
    13
af3eda069078 Added coding style document.
Florian Pose <fp@igh-essen.com>
parents:
diff changeset
    14
* No whitespace shall be left at the end of a line.
af3eda069078 Added coding style document.
Florian Pose <fp@igh-essen.com>
parents:
diff changeset
    15
af3eda069078 Added coding style document.
Florian Pose <fp@igh-essen.com>
parents:
diff changeset
    16
* After a comma, a single space shall be placed (if not followed by a
af3eda069078 Added coding style document.
Florian Pose <fp@igh-essen.com>
parents:
diff changeset
    17
  line break).
af3eda069078 Added coding style document.
Florian Pose <fp@igh-essen.com>
parents:
diff changeset
    18
af3eda069078 Added coding style document.
Florian Pose <fp@igh-essen.com>
parents:
diff changeset
    19
* Braces shall be placed in the following way (K&R style):
af3eda069078 Added coding style document.
Florian Pose <fp@igh-essen.com>
parents:
diff changeset
    20
af3eda069078 Added coding style document.
Florian Pose <fp@igh-essen.com>
parents:
diff changeset
    21
    if (...) {
af3eda069078 Added coding style document.
Florian Pose <fp@igh-essen.com>
parents:
diff changeset
    22
        ...
af3eda069078 Added coding style document.
Florian Pose <fp@igh-essen.com>
parents:
diff changeset
    23
    } else if (...) {
af3eda069078 Added coding style document.
Florian Pose <fp@igh-essen.com>
parents:
diff changeset
    24
        ...
af3eda069078 Added coding style document.
Florian Pose <fp@igh-essen.com>
parents:
diff changeset
    25
    } else {
af3eda069078 Added coding style document.
Florian Pose <fp@igh-essen.com>
parents:
diff changeset
    26
        ...
af3eda069078 Added coding style document.
Florian Pose <fp@igh-essen.com>
parents:
diff changeset
    27
    }
af3eda069078 Added coding style document.
Florian Pose <fp@igh-essen.com>
parents:
diff changeset
    28
af3eda069078 Added coding style document.
Florian Pose <fp@igh-essen.com>
parents:
diff changeset
    29
    int function(...)
af3eda069078 Added coding style document.
Florian Pose <fp@igh-essen.com>
parents:
diff changeset
    30
    {
af3eda069078 Added coding style document.
Florian Pose <fp@igh-essen.com>
parents:
diff changeset
    31
        ...
af3eda069078 Added coding style document.
Florian Pose <fp@igh-essen.com>
parents:
diff changeset
    32
    }
af3eda069078 Added coding style document.
Florian Pose <fp@igh-essen.com>
parents:
diff changeset
    33
af3eda069078 Added coding style document.
Florian Pose <fp@igh-essen.com>
parents:
diff changeset
    34
* Macros shall be named in CAPITAL letters. If a macro contains multiple
af3eda069078 Added coding style document.
Florian Pose <fp@igh-essen.com>
parents:
diff changeset
    35
  statements, they should be enclosed by a 'do {} while (0)' loop. Macro
af3eda069078 Added coding style document.
Florian Pose <fp@igh-essen.com>
parents:
diff changeset
    36
  parameters shall also be capital letters and shall be enclosed py parantheses
af3eda069078 Added coding style document.
Florian Pose <fp@igh-essen.com>
parents:
diff changeset
    37
  if necessary.
af3eda069078 Added coding style document.
Florian Pose <fp@igh-essen.com>
parents:
diff changeset
    38
af3eda069078 Added coding style document.
Florian Pose <fp@igh-essen.com>
parents:
diff changeset
    39
    #define MACRO(A, B) \
af3eda069078 Added coding style document.
Florian Pose <fp@igh-essen.com>
parents:
diff changeset
    40
        do { \
af3eda069078 Added coding style document.
Florian Pose <fp@igh-essen.com>
parents:
diff changeset
    41
            if ((A) == 1) { \
af3eda069078 Added coding style document.
Florian Pose <fp@igh-essen.com>
parents:
diff changeset
    42
                statement(B); \
af3eda069078 Added coding style document.
Florian Pose <fp@igh-essen.com>
parents:
diff changeset
    43
        } while (0)
af3eda069078 Added coding style document.
Florian Pose <fp@igh-essen.com>
parents:
diff changeset
    44