style nit: you can put the argument list on the previous line and only the body indent 2 chars