And on an even smaller sidenote:

i usually prefer having a blank line between the subroutine body and the return call so that:
a) know exactly where it's returning, easy for maintenance
b) know exactly what it's returning, easy for debugging

These are just my preferences...

jynx