[Webkit-unassigned] [Bug 104366] Make order iterator member stack allocated in RenderFlexibleBox

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=104366





--- Comment #10 from Carlos Garcia Campos <cgarcia at igalia.com>  2012-12-08 04:55:10 PST ---
(In reply to comment #7)
> I don't think Carlos is saying that avoiding heap allocations during layout is a rule.  He's just saying for this code, we'll avoid a heap allocation since we always create this object.  He is simply describing the change.

Yes, exactly, the iterator was not a member at first, then it became a member and Tony suggested that maybe we could make it stack allocated, which I think makes sense, so I wrote the patch. This doesn't try to fix a performance issue in the layout algorithm.

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