Looks sensible to me, but misses braces to comply with the coding conventions.