On Mon, 3 Dec 2001, Juergen Vigna wrote:
> On 03-Dec-2001 John Levon wrote:
> > yup, if you can explain :
> >
> > 843 if (!the_locking_inset) {
> > 844 bool found = lt->updateInset(bv, inset);
> > 845 if (clear)
> > 846 lt = 0;
> > 847 if (found)
> > 848 setUpdateStatus(bv, NONE);
> > 849 return found;
> > 850 }
> >
> > what if the inset changes size such that the containing inset needs to change
> > size too ? How does the resize of the insettext + insetcollapsable get triggered
> > now ?
>
> LyXText::updateInset does a rebreak and also redo the height of the row.
> You probably are confused by the NONE but you should have a look at the
> setUpdateStatus() functions which setts need_update to FULL if the LyXText
> requests this!
So you are saying that setUpdateStatus(NONE) indicates that no update
has been done so a FULL update is needed?
This convoluted logic seems a bit silly -- John and I have read
setUpdateStatus() as setUpdateRequirement() and given the context it
would seem more sensible to actually say how much of update we require
rather than how much has been done.
Allan. (ARRae)
