Peer review check-list
Syntax
No unnecessary blank lines
Blank lines do not contain spaces
Variables are named correctly (all lower case, no underscores)
Functions are named correctly
Output
Doesn't use buffered output unless absolutely necessary
Lang-strings used for output of text
No hard-coded strings for text output
Misc
Doesn't use deprecated functions https://docs.moodle.org/dev/Deprecated_functions_in_2.0
Code makes sense in the broader scheme of things – look at the whole function, not just the altered code.
If there is SQL code you can check quickly, do so.
Minimal DB calls (no excessive use of the DB).