Control Flow Related:
1.Are all the conditional paths reachable?
2.Are all the individual conditions in a complex conditions separately evaluated?
3.If there is a nested IF statement, are the THEN and ELSE parts appropriately delimited?
4. In the case of a multi-way branch like SWITCH/CASE statement, is a default clause provided? Are the breaks after each CASE appropriates?
5.Is there any part of code that is unreachable?
6.Are there any loops that will never execute?
7.Are there any loops where the final condition will never be met and hence cause the program to go into an infinite loop?
8.What is the level of nesting of the conditional statements? Can the code be simplified to reduce complexity?
1.Does the code follow the coding conventions of the organization?
2.Does the code follow any coding conventions that are platform specifics?
1.Are unhealthy programming constructs being used in the program?
2.Is there usage of specific idiosyncrasies of a particular machine architecture or a given version of an underlying product?
3. Is sufficient attention being paid to readability issues like indentation of code?
1.Have you checked for memory leaks?
1.Is the code adequately documented, especially where the logic is complex or the section of code is critical for product functioning?
2.Is appropriate change history documented?
3.Are the interfaces and the parameters thereof properly documented?