|
version 1.2, 2001/12/22 01:40:19
|
version 1.13, 2001/12/28 23:08:02
|
|
|
|
| if (condition) { | 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. |
| |
|
| |
7. All defined constants must begin with "PEGASUS_". |
| |
|
| |
8. All method must use the form myMethodName(). Many places |
| |
have my_method_name() (with underscores). |
| |
|
| |
9. Avoid use of (void). Use () instead. |
| |
|
| |
10. Either get rid of throw() declarations or use them |
| |
everywhere (perhaps they are not supported by some compilers). |
| |
|
| |
11. Avoid indenting "private:", "public:" and "protected". Use this |
| |
style: |
| |
|
| |
class X |
| |
{ |
| |
private: |
| |
... |
| |
public: |
| |
}; |
| |
|
| |
12. Use "Type* x" rather than "Type *x". |
| |
|
| |
13. Use "if (" rather than "if(". |
| |
|
| |
14. Avoid use of spaces around the condition: |
| |
|
| |
if( cond ) |
| |
|
| |
15. Keep all lines within 80 columns. |
| |
|
| |
16. Build fails on many Linux system (threading support problems). |
| |
|
| |
17. SSL is the default on Linux (is this okay?). |
| |
|
| |
19. Fix main logic (Unix v.s. Windows). |
| |
|
| |
21. Get rid of CIMIndication class. |
| |
|
| |
22. Revise licensing headers on all files. |
| |
|
| |
23. Add Log CVS directives. |
| |
|
| |
24. Avoid inlining of large functions. |
| |
|
| |
25. Find decent code formatter. |
| |
|
| |
26. Revisit use of bit masks in provider interfaces. |
| |
|
| |
27. Revisit provider loading/manager etc. |
| |
|
| |
28. Consolidate CIMOM handles. |
| |
|
| |
29. Get rid of directories containing .old extensions. |
| |
|
| |
30. Use constant global queue ids as described in MessagQueue.h |
| |
docs. |
| |
|
| |
31. Is the Facade class necessary. |
| |
|
| |
32. Response handler need not have to action methods (for single |
| |
and multiple objects). |
| |
|
| |
33. Clean up warnings on NT. |
| |
|
| |
34. Use Linkage.h files everywhere. |
| |
|
| |
35. Need to add notifiers (using message queue observer scheme). |
| |
|
| |
36. Need to define services's interface to CIMOM. |
| |
|
| |
37. Finalize the client interface (SDK). |
| |
|
Return to
CodeReviewNotes.txt
CVS log
Up to [Pegasus] / pegasus / doc / Attic