Hi all,
I have been reviewing a lot of code recently that has if statements that are failing to surround the block with brackets:
if (condition)
do something
else
do something else
A reminder that the agreed convention was to always use brackets. The reason I bring this up is that a few of the commits i have reviewed have been accompanied with whitespace formatting issues (using tabs rather than spaces). The two of these mishaps together basically make code unreadable.
Sorry to be a code formatting stickler on this one. But hey i had to grudgingly give up my "function( arg1, arg2 )" habit 
-Justin
--
Justin Deoliveira
OpenGeo - http://opengeo.org
Enterprise support for open source geospatial.