class Length merge
Dirk Mueller
mueller at kde.org
Sat Oct 18 03:51:09 CEST 2003
Hi,
The attached patch is a diff webcore / khtml tree and adds accessors and
proper encapsulation to the Length class. This is for several months without
problems or regressions in our tree, and it would diffing both trees
significantly easier when this would be merged by you. It also optimizes the
common construction case by providing a fast path for initializing all member
variables.
Its not a complete patch, but all other places that need fixing can be found
by the compiler. For example code like
if(foo.type == Variable)
is to be replaced with
if (foo.isVariable())
and code like
m_width = width.value;
is to be replaced with
m_width = width.value();
I'm looking forward to see this merged in the Apple tree.
--
> Looking for a KDE-related EMail-Alias ? Get one at kdemail.net for FREE! <
-------------- next part --------------
A non-text attachment was scrubbed...
Name: khtmllayout.diff
Type: text/x-diff
Size: 3191 bytes
Desc: not available
Url : http://mail.kde.org/mailman/private/khtml-devel/attachments/20031018/2808864c/khtmllayout.bin
More information about the Khtml-devel
mailing list