pegasus/doc/Attic/CodeReviewNotes.txt - diff

Return to CodeReviewNotes.txt CVS log

Up to [Pegasus] / pegasus / doc / Attic

version 1.6, 2001/12/25 04:41:18

version 1.12, 2001/12/28 17:15:37

Line 57

if( cond )

15. Changed "Permisyion" to "Permission"

15. Keep all lines within 80 columns.

16. Keep all lines within 80 columns.

16. Build fails on many Linux system (threading support problems).

17. Build seems to fail on many Linux system (threading problems).

17. SSL is the default on Linux (is this okay?).

18. SSL is the default on Linux (should not be used by defualt).

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.

19. Fix main logic (Unix v.s. Linux).

Legend: