[Webkit-unassigned] [Bug 15148] Poor performance on crazy DOM raytracer

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Sat Sep 8 05:31:57 PDT 2007


http://bugs.webkit.org/show_bug.cgi?id=15148





------- Comment #3 from mjs at apple.com  2007-09-08 05:31 PDT -------
This seems like a gratuitous change:

-        for ( ; (r = it.current()); ++it) {
+        for ( ; (r = it.current()); ++it ) {

There's no need to use new here, you can just make the Vector a local variable:

+    Vector<RenderObject*>* deadObjects = new Vector<RenderObject*>();

In fact, it might be worthwhile giving it an inline capacity to handle the
common case effeciently. Something like:

Vector<RenderObject*, 16> deadObjects;

r- to address these points.


-- 
Configure bugmail: http://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.



More information about the webkit-unassigned mailing list