With junior programmers I typically just look for high level patterns that are commonly wrong. Sure if they are touching our cross thread communications code I need to spend a lot of time on that because it is so complex nobody gets it right - but we only have a tiny amount of that and most people look at it and run the opposite way (even me - I wrote it but I still do my best not to touch it when I can avoid it - that is hard hard hard)
With junior programmers I typically just look for high level patterns that are commonly wrong. Sure if they are touching our cross thread communications code I need to spend a lot of time on that because it is so complex nobody gets it right - but we only have a tiny amount of that and most people look at it and run the opposite way (even me - I wrote it but I still do my best not to touch it when I can avoid it - that is hard hard hard)