[Patch] inline boxes leak

Germain Garand germain at ebooksfrance.org
Mon Jun 21 04:43:04 BST 2004


Hello,

Here is a patch intended to solve the problem of leaked placeholder boxes [1].

It replaces the deleteLineBoxes/deleteTextBoxes method with a single virtual 
named deleteInlineBoxes, that is the exact parallel to the 
createInlineBoxes method.

Sadly, each inline needing to be cleared by it's associated RenderObject at 
relayout time, I couldn't avoid a new member in RenderBox.

Also in the patch: a simplification of the Bidinext logic (no need to return 
each inline twice).

What do you think about it?

Germain

[1] Inline boxes acting as a "black box" container for replaced and positioned 
elements. They used to be merely detached just after layout, corrupting the 
inline tree in the process.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: inlineboxes-leak.diff
Type: text/x-diff
Size: 12119 bytes
Desc: not available
URL: <https://mail.kde.org/mailman/private/kfm-devel/attachments/20040621/55ecdeac/attachment.diff>


More information about the kfm-devel mailing list