1 mike 1.1
2
|
3 mike 1.2 1. Indent four spaces (not three and not tabs).
|
4 mike 1.1
5 2. Literals used (other than 0 or 1).
6
7 3. NULL used in several places.
|
8 mike 1.2
9 4. Put braces on their own lines like this:
10
11 if (condition)
12 {
13
14 }
15
16 Avoid this style:
17
18 if (condition) {
19
20 }
|
21 mike 1.3
22 5. Makedepend utility was made the default depend
23 utility on some platforms. The deafault should be
24 "mu depend".
|
25 mike 1.4
26 6. Regression tests should only print this:
27
28 "+++++ passed all tests"
29
30 Extraneous output should be removed.
|
31 mike 1.5
32 7. All defined constants must begin with "PEGASUS_".
33
34 8. All method must use the form myMethodName(). Many places
35 have my_method_name() (with underscores).
36
37 9. Avoid use of (void). Use () instead.
38
39 10. Either get rid of throw() declarations or use them
40 everywhere (perhaps they are not supported by some compilers).
41
42 11. Avoid indenting "private:", "public:" and "protected". Use this
43 style:
44
45 class X
46 {
47 private:
48 ...
49 public:
50 };
51
52 mike 1.5 12. Use "Type* x" rather than "Type *x".
53
54 13. Use "if (" rather than "if(".
55
56 14. Avoid use of spaces around the condition:
57
58 if( cond )
59
60 15. Changed "Permisyion" to "Permission"
61
62 16. Keep all lines within 80 columns.
|