The diff makes it look like the while is indented too far.  If you view the file directly it's not.  Weirded me out when I first put the pull request together.