On Mon, Jul 23, 2012 at 06:47:34PM +0200, Miklos Vajna wrote: > On Mon, Jul 23, 2012 at 05:59:29PM +0200, Michael Stahl <[email protected]> > 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.
+1 and pushed. D. _______________________________________________ LibreOffice mailing list [email protected] http://lists.freedesktop.org/mailman/listinfo/libreoffice
