https://bugs.freedesktop.org/show_bug.cgi?id=66025

--- Comment #36 from David Bolen <[email protected]> ---
Created attachment 82905
  --> https://bugs.freedesktop.org/attachment.cgi?id=82905&action=edit
Patch correction

Hmm, this is embarrassing.  In seeing the patch committed, I suddenly realized
I left one bit of test code in it, in the first exception handler (which has no
need to assign the exception to 'e').  This patch just removes that small bit
of code, and applies on top of the prior patch.

The original patch technically works fine as-is, and I considered not even
bothering with a fix, but some future reviewer may wonder about any
relationship between that variable and the final exception (there isn't any)
due to their naming, so it feels worth cleaning up.

My apologies.  I don't know how I missed it originally.

-- 
You are receiving this mail because:
You are the assignee for the bug.
_______________________________________________
Libreoffice-bugs mailing list
[email protected]
http://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs

Reply via email to