Alfredo Braunstein wrote: > This patch sanitizes fitCursor. > > The main trick is that all y information (but not x) is known after the > first drawing step, and this y information is the only one needed for > fitCursor. So we simply do a redoParagraph on the outer paragraph of the > cursor before asking for it. > > Aditionally: > > > + remove some unneeded update calls > + remove LCursor::cached_y_ and LCursor::updatePos. Not needed nor used > anymore. > + remove some outdated comments
Ok, I've applied this and then more testing revealed problems, so I've just reverted it (too much beer probably). The fact is that after a redoParagraph (in the outer inset) we *should* have the correct y information for insets, but we don't have it now. For that we need to put the metrics call again inside the rebreak loop (as in some patch I've sent to the list some time ago). I'll try to do both and test it better this time. Regards, Alfredo
