Our coding conventions do not explicitly mention it to have function body on a separate line, but I think we still should do it for consistency. You're already doing it in this file except this particular instance.