summaryrefslogtreecommitdiff
path: root/doc
diff options
context:
space:
mode:
authoreinhverfr <einhverfr@4979c152-3d1c-0410-bac9-87ea11338e46>2006-09-09 06:55:13 +0000
committereinhverfr <einhverfr@4979c152-3d1c-0410-bac9-87ea11338e46>2006-09-09 06:55:13 +0000
commit43629eb68af5e8af74b72576f56f359bf975bd07 (patch)
treec1b4af7b8d0d915b79e4b9276e8df079398078b9 /doc
parent1036b3068092e00d59c21c4538fc841a1a24b1a0 (diff)
Added coding standard
git-svn-id: https://ledger-smb.svn.sourceforge.net/svnroot/ledger-smb/trunk@44 4979c152-3d1c-0410-bac9-87ea11338e46
Diffstat (limited to 'doc')
-rw-r--r--doc/coding-standard.pdfbin0 -> 44552 bytes
-rw-r--r--doc/coding-standard.tex66
2 files changed, 66 insertions, 0 deletions
diff --git a/doc/coding-standard.pdf b/doc/coding-standard.pdf
new file mode 100644
index 00000000..ec407ccf
--- /dev/null
+++ b/doc/coding-standard.pdf
Binary files differ
diff --git a/doc/coding-standard.tex b/doc/coding-standard.tex
new file mode 100644
index 00000000..424dbe24
--- /dev/null
+++ b/doc/coding-standard.tex
@@ -0,0 +1,66 @@
+\documentclass{article}
+\usepackage{metatron}
+\title{LedgerSMB Coding Standards}
+\author{The LedgerSMB Development Team}
+\begin{document}
+\maketitle
+
+\tableofcontents
+
+\section{Introduction}
+Consistant coding style contributes to readability of code. This contributes in
+turn to fewer bugs and easier debugging.
+
+While consultants which implement custom solutions for their customers are not
+required to follow these coding guidelines, doing so will likely contribute to
+efficiency of the project and, if desired, the likelihood of having the changes
+accepted in the main source tree.
+
+\section{File Organization and Whitespace}
+
+Files should be organized according to three principles: atomicity, performance,
+and readibility. While there is no maximum file size, there is no reason to
+group large amounts of code together when only a small subset of that code will
+be executed on any given run of the program. Similarly, core API for a single
+data structure entity may be grouped together even if each run may touch only a
+small part of the code so that these functions can be maintained together in a
+logical way.
+
+Nested blocks of code should be indented with a single tab. This way,
+programmers can adjust the tab width of their editors to their preferences.
+
+Lines longer than 79 characters are not handled well by many terminals and
+should generally be avoided. Continued lines should be indented by one more tab
+than either the line above or below (i.e. if the line above is indented by two
+tabs and the line below by three, indent the continued segment by four).
+
+Lines longer than 79 characters cause problems in some terminals and should be
+avoided.
+
+\section{Comments}
+
+In an ideal world, the code should be sufficiently readible to be entirely
+understandable without comments. Unfortunately such an ideal is rarely
+attained. Comments should be used to annotate code and add information that is
+not already a part of code or programming logic itself.
+
+Comments may include arguments and return values of functions (for easy
+reference), a summary as to why a particular design choice was made, or a note
+to alert future programmers that a particular chunk of code deserves additional
+attention. Comments should not, however generaly tell what the program is doing
+or how it does that. If such comments are required, it is a good indication
+that a block of code requires rewriting.
+
+Additionally it is a good idea to provide a brief description at the top of each
+file describing, in general terms, what its function is.
+
+\section{Function Organization}
+
+Functions should be atomic. While there is no maximum length to functions, long
+functions may be an indication that a function may be non-atomic.
+
+In general, when more than one line of code is being copied and pasted, it
+should instead be moved into its own function where it can be called by all
+entry points.
+
+\end{document}