Review Request: use member initialization lists in khtml
Jaime Torres
jtamate at gmail.com
Sat Oct 17 10:32:48 BST 2009
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://reviewboard.kde.org/r/1876/
-----------------------------------------------------------
Review request for kdelibs.
Summary
-------
Make use of the member initialization lists in the constructors of the header files.
The red spaces are already removed.
Diffs
-----
/trunk/KDE/kdelibs/khtml/css/css_base.h 1036373
/trunk/KDE/kdelibs/khtml/dom/QualifiedName.h 1036373
/trunk/KDE/kdelibs/khtml/dom/css_stylesheet.h 1036373
/trunk/KDE/kdelibs/khtml/dom/dom2_range.h 1036373
/trunk/KDE/kdelibs/khtml/dom/dom_exception.h 1036373
/trunk/KDE/kdelibs/khtml/ecma/kjs_proxy.h 1036373
/trunk/KDE/kdelibs/khtml/html/htmltokenizer.h 1036373
/trunk/KDE/kdelibs/khtml/khtmlpart_p.h 1036373
/trunk/KDE/kdelibs/khtml/misc/AtomicString.h 1036373
/trunk/KDE/kdelibs/khtml/misc/idstring.h 1036373
/trunk/KDE/kdelibs/khtml/misc/loader.h 1036373
/trunk/KDE/kdelibs/khtml/misc/paintbuffer.h 1036373
/trunk/KDE/kdelibs/khtml/misc/shared.h 1036373
/trunk/KDE/kdelibs/khtml/platform/graphics/IntRect.h 1036373
/trunk/KDE/kdelibs/khtml/rendering/bidi.h 1036373
/trunk/KDE/kdelibs/khtml/rendering/render_block.h 1036373
/trunk/KDE/kdelibs/khtml/rendering/render_canvas.h 1036373
/trunk/KDE/kdelibs/khtml/rendering/render_flow.h 1036373
/trunk/KDE/kdelibs/khtml/rendering/render_line.h 1036373
/trunk/KDE/kdelibs/khtml/rendering/render_position.h 1036373
/trunk/KDE/kdelibs/khtml/rendering/render_replaced.h 1036373
/trunk/KDE/kdelibs/khtml/rendering/render_style.h 1036373
/trunk/KDE/kdelibs/khtml/svg/SVGGlyphElement.h 1036373
/trunk/KDE/kdelibs/khtml/xml/ClassNames.h 1036373
/trunk/KDE/kdelibs/khtml/xml/dom_nodeimpl.h 1036373
/trunk/KDE/kdelibs/khtml/xml/dom_nodelistimpl.h 1036373
/trunk/KDE/kdelibs/khtml/xml/dom_stringimpl.h 1036373
Diff: http://reviewboard.kde.org/r/1876/diff
Testing
-------
Everything I've seen keeps working as before.
The acid2 test gets 100% with testkhtml, but still 80% with konqueror in a 64bits environment.
Thanks,
Jaime
More information about the kde-core-devel
mailing list