I'm still seeing code like:
if (foo) {
doSomething();
}
Please don't use K & R style, that's not our project style. Don't do it!
I must have said this many times now, I'm not sure why it's not sinking in.
Also don't do this:
if (x)
doSomething();
Also please don't paste public methods into the private areas of files. Respect the different areas of files.
Next time I see code like the above I'm just going to delete it :)