49808b8f1c
- mention Uncrustify instead of GNU Indent - add a missing argument of code_style_example_function() - make the function return 0 or 1 as per doxygen comment - replace tab with spaces - correct case() to switch() in a comment - add an indentation example for switch statement
137 lines
4.4 KiB
C
137 lines
4.4 KiB
C
/**
|
|
* \defgroup coding-style Coding style
|
|
*
|
|
* This is how a Doxygen module is documented - start with a \defgroup
|
|
* Doxygen keyword at the beginning of the file to define a module,
|
|
* and use the \addtogroup Doxygen keyword in all other files that
|
|
* belong to the same module. Typically, the \defgroup is placed in
|
|
* the .h file and \addtogroup in the .c file.
|
|
*
|
|
* The Contiki source code contains an Uncrustify configuration file,
|
|
* uncrustify.cfg, under tools/code-style and small helper scripts are
|
|
* provided at the same place. Note that this is not a silver bullet -
|
|
* for example, the script does not add separators between functions,
|
|
* nor does it format comments correctly. The script should be treated
|
|
* as an aid in formatting code: first run the formatter on the source
|
|
* code, then manually edit the file.
|
|
*
|
|
* @{
|
|
*/
|
|
|
|
/**
|
|
* \file
|
|
* A brief description of what this file is.
|
|
* \author
|
|
* Adam Dunkels <adam@dunkels.com>
|
|
*
|
|
* Every file that is part of a documented module has to have
|
|
* a \file block, else it will not show up in the Doxygen
|
|
* "Modules" * section.
|
|
*/
|
|
|
|
/* Single line comments look like this. */
|
|
|
|
/*
|
|
* Multi-line comments look like this. Comments should prefferably be
|
|
* full sentences, filled to look like real paragraphs.
|
|
*/
|
|
|
|
#include "contiki.h"
|
|
|
|
/*
|
|
* Make sure that non-global variables are all maked with the static
|
|
* keyword. This keeps the size of the symbol table down.
|
|
*/
|
|
static int flag;
|
|
|
|
/*
|
|
* All variables and functions that are visible outside of the file
|
|
* should have the module name prepended to them. This makes it easy
|
|
* to know where to look for function and variable definitions.
|
|
*
|
|
* Put dividers (a single-line comment consisting only of dashes)
|
|
* between functions.
|
|
*/
|
|
/*---------------------------------------------------------------------------*/
|
|
/**
|
|
* \brief Use Doxygen documentation for functions.
|
|
* \param c Briefly describe all parameters.
|
|
* \return Briefly describe the return value.
|
|
* \retval 0 Functions that return a few specified values
|
|
* \retval 1 can use the \retval keyword instead of \return.
|
|
*
|
|
* Put a longer description of what the function does
|
|
* after the preamble of Doxygen keywords.
|
|
*
|
|
* This template should always be used to document
|
|
* functions. The text following the introduction is used
|
|
* as the function's documentation.
|
|
*
|
|
* Function prototypes have the return type on one line,
|
|
* the name and arguments on one line (with no space
|
|
* between the name and the first parenthesis), followed
|
|
* by a single curly bracket on its own line.
|
|
*/
|
|
int
|
|
code_style_example_function(char c)
|
|
{
|
|
/*
|
|
* Local variables should always be declared at the start of the
|
|
* function.
|
|
*/
|
|
int i; /* Use short variable names for loop
|
|
counters. */
|
|
|
|
/*
|
|
* There should be no space between keywords and the first
|
|
* parenthesis. There should be spaces around binary operators, no
|
|
* spaces between a unary operator and its operand.
|
|
*
|
|
* Curly brackets following for(), if(), do, and switch() statements
|
|
* should follow the statement on the same line.
|
|
*/
|
|
for(i = 0; i < 10; ++i) {
|
|
/*
|
|
* Always use full blocks (curly brackets) after if(), for(), and
|
|
* while() statements, even though the statement is a single line
|
|
* of code. This makes the code easier to read and modifications
|
|
* are less error prone.
|
|
*/
|
|
if(i == c) {
|
|
return 1; /* No parentesis around return values. */
|
|
} else { /* The else keyword is placed inbetween
|
|
curly brackers, always on its own line. */
|
|
c++;
|
|
}
|
|
}
|
|
|
|
/*
|
|
* Indent "case" statement and "default" label by two spaces in "switch"
|
|
* statement.
|
|
*/
|
|
switch(c) {
|
|
case 19:
|
|
return 1;
|
|
default:
|
|
break;
|
|
}
|
|
|
|
return 0;
|
|
}
|
|
/*---------------------------------------------------------------------------*/
|
|
/*
|
|
* Static (non-global) functions do not need Doxygen comments. The
|
|
* name should not be prepended with the module name - doing so would
|
|
* create confusion.
|
|
*/
|
|
static void
|
|
an_example_function(void)
|
|
{
|
|
|
|
}
|
|
/*---------------------------------------------------------------------------*/
|
|
|
|
/* The following stuff ends the \defgroup block at the beginning of
|
|
the file: */
|
|
|
|
/** @} */
|