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