On Mon, Jul 23, 2012 at 05:59:29PM +0200, Michael Stahl <mst...@redhat.com> 
wrote:
> i was wondering why you'd removed the content of the then branch until i
> realized that that is actually error handling code for a failed close
> call... it's really painful to read code with 2 side effects in the if
> clause....

+1 from me, one more needed.
_______________________________________________
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice

Reply via email to