|
version 1.1, 2001/12/22 00:45:38
|
version 1.4, 2001/12/22 02:06:41
|
|
|
|
| | |
| | |
| 1. The indentation standard is four. |
1. Indent four spaces (not three and not tabs). |
| | |
| 2. Literals used (other than 0 or 1). | 2. Literals used (other than 0 or 1). |
| | |
| 3. NULL used in several places. | 3. NULL used in several places. |
| |
|
| |
4. Put braces on their own lines like this: |
| |
|
| |
if (condition) |
| |
{ |
| |
|
| |
} |
| |
|
| |
Avoid this style: |
| |
|
| |
if (condition) { |
| |
|
| |
} |
| |
|
| |
5. Makedepend utility was made the default depend |
| |
utility on some platforms. The deafault should be |
| |
"mu depend". |
| |
|
| |
6. Regression tests should only print this: |
| |
|
| |
"+++++ passed all tests" |
| |
|
| |
Extraneous output should be removed. |
Return to
CodeReviewNotes.txt
CVS log
Up to [Pegasus] / pegasus / doc / Attic