Created attachment 24712 [details] Patch for #47424 This patch solves the problem of an infinite loop in footnotes as reported in bug 47424. The infinite loop occurred in org.apache.fop.layoutmgr.PageBreakingAlgorithm.getFootnoteSplit(int, int, int, int, boolean). This patch does not solve the problem of another infinite loop in footnotes as reported in bugs 48063 and 48162. This infinite loop occurs in org.apache.fop.layoutmgr.PageBreakingAlgorithm.createFootnotePages(KnuthPageNode). The test attached to 48063 is converted to a testcase and added to the list of disabled testcases.
Hi Heidi, Thanks for your patch. Although I'm afraid it's more of a workaround than an actual fix for the issue. Not that it's useless, it may actually help people who run into this issue and need a quick temporary solution. But I'm not sure we want it to be integrated into the code. I acknowledge that page breaking is not the simplest piece of code, but it would be good to identify the origin of the error. From a quick debugging session that I made some time ago, I noticed errors in the PageBreakingAlgorithm.restartFrom method: some footnote-related variables are not properly reset, which means that when getFootnoteSplit is called it's believed to be more footnote content than there actually is. The problem may lie there. Creating a simple test case is key to help the debugging. I would try to create an FO file that is just a few lines high, with no side region (region-before, -after, etc.) and specify every dimension in points (to have more 'even' values in the debugger). Then, define the page height such that the content doesn't exactly fit (for example, 105pt high with lines of 10pt), which will trigger the call to the restartFrom method, and start debugging from there. The second attachment to bug 47424 may actually be a good start. ATM I don't have much time to look into this issue in more detail unfortunately. I hope those few notes will help you get started if you want to have a go at it. Meanwhile, your patch will be helpful to people in urgent need for a workaround. Thanks, Vincent (In reply to comment #0) > Created an attachment (id=24712) [details] > Patch for #47424 > > This patch solves the problem of an infinite loop in footnotes as reported in > bug 47424. > > The infinite loop occurred in > org.apache.fop.layoutmgr.PageBreakingAlgorithm.getFootnoteSplit(int, int, int, > int, boolean). > > This patch does not solve the problem of another infinite loop in footnotes as > reported in bugs 48063 and 48162. This infinite loop occurs in > org.apache.fop.layoutmgr.PageBreakingAlgorithm.createFootnotePages(KnuthPageNode). > The test attached to 48063 is converted to a testcase and added to the list of > disabled testcases.
resetting P2 open bugs to P3 pending further review
increase priority for bugs with a patch