I know, it's not the point of this commit, but using until f.eof is confusing since the loop is only expected to execute once since `f.read` will read the whole file.  It would seem better to use `unless f.eof` here to show the code intent better.