On Thu, Jun 29, 2006 at 01:08:32AM +0300, Martin Vermeer wrote: > On Wed, Jun 28, 2006 at 03:06:36PM +0200, Jean-Marc Lasgouttes wrote: > > >>>>> "Jean-Marc" == Jean-Marc Lasgouttes <[EMAIL PROTECTED]> writes: > > > > Jean-Marc> This is because the rowpainter code sets Wide() on these > > Jean-Marc> insets, because they are alone on a row. The following > > Jean-Marc> patches test additionally that the inset is not the last > > Jean-Marc> element of the paragraph. It seems to work for me, but I am > > Jean-Marc> not sure it is the right condition to test. > > No, I have serious doubts here. And there will be situations where an > inset is the last (or only) element of a paragraph, and contains lots of > texts that we want to render parsimoniously. So I am afraid that this > subverts the speed-up effect.
I enabled the painting debugging by -dbg PAINTING, and can confirm my doubts. A Note inset, even if apparently alone on its row and followed by text in the same paragraph that has wrapped to the next row, will _never_ go Wide with your patch. (I tried the 1.4 version). I expect Bennett would find the old slowness back. Yes, it "fixes" -- rather, suppresses -- the note-in-tabular bug :-( - Martin
pgp3yyL59YgxX.pgp
Description: PGP signature
