It's better to write 'if' at a new line, i.e.:
```cpp
if ( inn.eof() )
    break;
```