[Libreoffice] [REVIEW] fdo#43965 fix RTF import of page break symbol

Miklos Vajna vmiklos at frugalware.org
Mon Jan 30 14:41:32 PST 2012

Hi Michael,

On Mon, Jan 30, 2012 at 10:07:50PM +0100, Michael Stahl <mstahl at redhat.com> wrote:
> > See
> > http://cgit.freedesktop.org/libreoffice/core/commit/?id=2aa3d43a3746c797391afb1a08c024d512976006
> seems that with this change not only the new LN_PFPageBreakBefore from
> \page is reset when a \par occurs, but also the LN_PFPageBreakBefore
> inserted from \pagebb.
> i have no idea if that is good or bad, but your comment says "since it's
> a symbol", which is only true for \page, not \pagebb; is it intentional?

Thanks for the catch, that's indeed not intentional and it is
definitely bad. Here is a followup commit:


I'm attaching a patch combining the two, to be pushed to -3-5.


More information about the LibreOffice mailing list