2012-10-10T08:06:50 Raubtier

Allerdings ist das -r mit folgendem open nicht ein atomarer Befehl, d.h. es kann zwischen dem -r und dem open irgendwas passieren, dass die Datei doch nicht mehr lesbar ist.

Das stimmt, ist aber wohl eher ein theoretisches Problem.
Wenn die Befehle im Programm direkt hintereinander stehen, dürfte es sogar schwierig sein, den Fall gezielt zu konstruieren. Und als Fallback hat man immer noch die Fehlermeldung aus open(). Der Returnwert wird im Beispiel ja korrekt überprüft.

2012-10-10T08:06:50 Raubtier

Auch würde ich das -s vermeiden, denn man möchte ggf. auch mal von STDIN oder einer Pipe lesen.

Der -s Check hat den Vorteil, dass man sich ggf. nicht wundern muss, warum die Ausgabe leer bleibt. Wenn man also ohnehin immer aus einer Datei lesen will, kann er ganz nützlich sein.