Could we actually kill the return here and have it read like:

```
if @eof
  nil
elsif @params[:nonblock]
```

I think that accomplishes the same thing, but is a bit more consistent in terms of style with other parts of the code base.