Looks sane to me. pushed to the 3-4 branch 2 more reviews for 3.4.2 needed <div class="gmail_quote">2011/7/14 Kohei Yoshida <<a href="mailto:kyoshida@novell.com">kyoshida@novell.com</a>> <blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">Hi there, I'd like to have the following commit: <a href="http://cgit.freedesktop.org/libreoffice/calc/commit/?id=cdb8cf074c8536aa3ea30102bcb9b34feda21833" target="_blank">http://cgit.freedesktop.org/libreoffice/calc/commit/?id=cdb8cf074c8536aa3ea30102bcb9b34feda21833</a> cherry-picked to the -3-4 as well as the -3-4-2 branch, in order to fix <a href="https://bugs.freedesktop.org/show_bug.cgi?id=36688" target="_blank">https://bugs.freedesktop.org/show_bug.cgi?id=36688</a> This is a simple, one-linear fix, and I consider this to be a very safe fix. Review, sign-off etc appreciated. Kohei -- Kohei Yoshida, LibreOffice hacker, Calc <<a href="mailto:kyoshida@novell.com">kyoshida@novell.com</a>> _______________________________________________ LibreOffice mailing list <a href="mailto:LibreOffice@lists.freedesktop.org">LibreOffice@lists.freedesktop.org</a> <a href="http://lists.freedesktop.org/mailman/listinfo/libreoffice" target="_blank">http://lists.freedesktop.org/mailman/listinfo/libreoffice</a> </blockquote></div>