On Mon, Feb 18, 2013 at 6:38 PM, Darin Adler <da...@apple.com> wrote:
> On Feb 18, 2013, at 6:28 PM, Ryosuke Niwa <rn...@webkit.org> wrote: > > > I had the same question when I found this class today but when I tried > to turn it into a function, I realized that each caller of this function > now needs to maintain two variables instead of one. Since those two > variables (node and distributedNode) need to be in a consistent state, I've > thought of creating a state object but then it seemed silly to create a > state object only and a free function. It's much more natural for it be > just a "walker" class. > > Does it? Is it really necessary to have the distributed node state or is > that just performance optimization? > Many users of this class ends up walking up the ancestry tree. In that case, having a state/iterator object like this makes sense. But you're right in that the case of one off call to parent(), there isn't much point in creating an object. We can probably get away with a single function call. - R. Niwa
_______________________________________________ webkit-dev mailing list webkit-dev@lists.webkit.org https://lists.webkit.org/mailman/listinfo/webkit-dev
