Looks sane and safe to me. Pushed with my sign-off to the 3-4 branch. <div class="gmail_quote">2011/7/7 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=6e1ff6d24f0c93af145a73b27918fe49b1960b71" target="_blank">http://cgit.freedesktop.org/libreoffice/calc/commit/?id=6e1ff6d24f0c93af145a73b27918fe49b1960b71</a> cherry-picked to the -3-4 branch. Excel only supports up to 1026 page breaks per sheet, and the above commit enforces that during export to xls. It's a very simple change with a very limited scope. Review & sign-off 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>