aboutsummaryrefslogtreecommitdiffstats
path: root/notes
diff options
context:
space:
mode:
authorMarti Bolivar <mbolivar@leaflabs.com>2011-04-06 11:51:57 -0400
committerMarti Bolivar <mbolivar@leaflabs.com>2011-04-06 11:51:57 -0400
commit012d9ba1c2069d4d072685b990cbd01b611bb751 (patch)
tree8cd4b6f80acf5d4a57b21ea1331904ed463d0699 /notes
parentb835b9864d7fa17de8eeaaafbb1b17f2252e68a5 (diff)
downloadlibrambutan-012d9ba1c2069d4d072685b990cbd01b611bb751.tar.gz
librambutan-012d9ba1c2069d4d072685b990cbd01b611bb751.zip
Updated coding standard.
Diffstat (limited to 'notes')
-rw-r--r--notes/coding_standard.txt14
1 files changed, 8 insertions, 6 deletions
diff --git a/notes/coding_standard.txt b/notes/coding_standard.txt
index b81847d..0accda7 100644
--- a/notes/coding_standard.txt
+++ b/notes/coding_standard.txt
@@ -122,7 +122,7 @@ Naming conventions
There's always a fight about upper and lower case vs. underscores.
We'll handle this as follows.
-- First, Dammit_Dont_Mix_Like_This, because It_Looks_Really_Ugly, ok?
+- First, Dont_Mix_Like_This, because It_Looks_Really_Ugly, ok?
[There's been some debate about this, and some exceptions are
already grandfathered in, so in order to settle it, let's call this
a "recommendation" instead of "requirement".]
@@ -140,11 +140,9 @@ We'll handle this as follows.
- Functions: C functions are all lowercase, and words are separated by
underscores. C++ method names are camel cased (thisIsAnExample).
-- Structs: pick a style from "Variables" or "Classes" depending on how
- you mean it (since it might be either a simple record type, in which
- case do like c variables, or you might be faking an object in c, in
- which case do like classes). If it's in a typedef, don't feel
- obliged to put "_t" at the end of the name; we don't.
+- Structs: Usually like variables (adc_dev, adc_reg_map, etc.), but
+ it's not crucial. If it's in a typedef, don't feel obliged to put
+ "_t" at the end of the name; we don't.
- Macros and constants: all caps, separated by underscores. Variables
with the "const" qualifier aren't considered "constants" for the
@@ -196,6 +194,10 @@ Documentation
maintenance burden of documenting things in two places at once, and
makes it easier for documentation to go stale.
+ If you do have to document something manually, put a comment in the
+ source file informing future maintainers about it, so they'll pay
+ extra attention when making changes.
+
- For libmaple proper (the pure C library under libmaple/); the
convention is to document any user-facing entity at the point where
it is defined. In particular, this means you should document an