Coffee returns the last executed expression in a function by default, so you end up with some unnecessary returns doing this sort of thing.

You're right - we never use the return value here, so remove the returns.