John Levon <[EMAIL PROTECTED]> writes:

| On Mon, Mar 17, 2003 at 12:54:40AM +0100, Lars Gullik Bj?nnes wrote:
| 
| > Please comment.
| 
| If bv_owner is always != 0, what is the isTopLevel() test all about ?

Just to be explicit.

It seems that either isTopLevel or isInInset can be removed.
 
| This seems dubious to me, look at RowPainter::paintFirst() - if bv_owner
| was always set, then we'd get the top margin at the top of every
| insettext (and we don't, so ...)

Note that the isTopLevel also checks that inset_owner == 0.
 
-- 
        Lgb

Reply via email to