This is a matter of personal preference, of course, but I find these blank lines disconcerting. I use blank lines in code as in prose: to separate ideas into logically related paragraphs. Lines 156, 157, 159, and 161 should be a single paragraph, from this perspective.