[Webkit-unassigned] [Bug 85844] Inline Node::traverseNextNode

[bugzilla-daemon at webkit.org]

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





--- Comment #8 from Ryosuke Niwa <rniwa at webkit.org>  2012-05-07 17:12:30 PST ---
(In reply to comment #7)
> (In reply to comment #6)
> > What I meant is that you could just call traverseNextNode() in that case. Oh well, I guess compilers are smart enough to move the condition out of the loop?
> 
> That still adds an extra branch for no reason. It should really be the callers responsibility to use the right variant as usually they just need one or the other.

Fair enough. I bet editing code is the one that calls this function with null stayWithin.

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