Tiny style nit, but for readability I always try to put a blank line btn the end of the function signature and the start of the function code when the signature spans multiple lines.