On Sat, Aug 10, 2002 at 08:00:08PM +0200, Lars Gullik Bjønnes wrote: > Please review this and comment.
I don't like this "de/increasePosAfterPos" thing, it's thoroughly confusing to name two different things pos. How about : void fixupPos(InsetList::iterator b, InsetList::iterator e, int amount) then : fixupPos(pos, end, 1) fixupPos(pos, end, -1) or even std::for_each(pos, end, fixupPos(-1)); it's not very nice that the inset table is loosely coupled with the paragraph in this respect but I don't think of a better solution right now. regards john -- "It is unbecoming for young men to utter maxims." - Aristotle