There are two blank lines between sections of code in this function. That's never really desirable. (it's a style nit)