Checklist for reviewers
This checklist must contain no more than 20 items. If item 21 is added remove the least significant item.
- There are no tabs.
- There are no trailing spaces.
- Javadocs are consistent with the code.
- Javadocs include the invariants (eg: null/not null for attributes).
- Javadocs include the preconditions (eg: null/not null for parameters).
- Check that all parameters are final.
- Make sure new clasess are created in the correct package.
- When reviewing a bug fix, make sure there is a test that checks for such bug.
- Make sure singleton classes do not hold references to non-singleton classes.