diff options
author | Thomas White <taw@physics.org> | 2011-04-01 17:51:01 +0200 |
---|---|---|
committer | Thomas White <taw@physics.org> | 2012-02-22 15:27:23 +0100 |
commit | 7906eea45cdfbe54be96d6bc99906e492d14f284 (patch) | |
tree | 5e480863f49bd83e41f0831f1fe7e603ef967fba /doc | |
parent | ee02a33acba401edb89d0abf5b4f52be79fe603b (diff) |
Update coding style
Diffstat (limited to 'doc')
-rw-r--r-- | doc/reference/xml/coding-standards.xml | 34 |
1 files changed, 33 insertions, 1 deletions
diff --git a/doc/reference/xml/coding-standards.xml b/doc/reference/xml/coding-standards.xml index e3d549f4..242719de 100644 --- a/doc/reference/xml/coding-standards.xml +++ b/doc/reference/xml/coding-standards.xml @@ -39,7 +39,22 @@ Whenever you edit a source file, don't forget to update the copyright dates at t <para> <emphasis>Indentation</emphasis> is done with <emphasis>tabs</emphasis> and <emphasis>alignment</emphasis> is done with spaces. This way, the code looks -neat whatever width you configure your editor to display tabs as. +neat whatever width you configure your editor to display tabs as. This means, +for example: +</para> +<para> +<programlisting> +struct something +{ + int thing; /* <--- spaces used to align comments */ + int thing_with_longer_name; /* <--- spaces used to align comments */ +} + +void somefunction(int something) +{ + /* <--- Tab character used at the start of this line */ +} +</programlisting> </para> <para> However, code must be <emphasis>strictly</emphasis> wrapped at 80 columns, or @@ -47,6 +62,23 @@ what would be 80 columns if the tabs were displayed as 8 spaces. If you think you need more width, you're at too many levels of indentation and need to break things down a bit. </para> +<para> +When performing a two or three dimensional iteration, for example over image +coordinates or Miller indices, it is acceptable to indent as follows: +</para> +<para> +<programlisting> +for ( h=-10; h<+10; h++ ) { +for ( k=-10; k<+10; k++ ) { +for ( l=-10; l<+10; l++ ) { + + /* Do stuff */ + +} +} +} +</programlisting> +</para> </refsect1> |