nit: you need braces around the if body