On Sep 1, 2010, at 7:04 PM, David Hyatt wrote: > We should just kill Arena and remove it and RenderArena both.
Wasn't it still a measurable slowdown last time we tried that? - Maciej > > On Sep 1, 2010, at 4:20 PM, Chris Marrin wrote: > >> >> Ken's PODRedBlackTree patch has made me go back and take a closer look at >> WebKit's Arena "class". Turns out it's not a class at all, just some structs >> and macros. That seems very un-WebKit-like to me. Ken's patch also has a >> PODArena class, which uses Arena in its implementation. Sam suggests that >> PODRedBlackTree should really go into WTF, which means PODArena and Arena >> would need to go there as well. >> >> It seems like Arena really needs to be brought into the 21st century and >> made a proper class. Maybe now is the right time to: >> >> 1) Make Arena a class >> >> 2) Integrate Ken's PODArena functionality into this new Arena class (or >> maybe just make Ken's PODArena the new Arena class). >> >> 3) Move the new Arena class to WTF >> >> 4) Put PODRedBlackTree in WTF >> >> It looks like RenderArena is currently the only client of Arena.h, so this >> change shouldn't be too hard. Of course, looking at RenderArena, it's a >> little odd, too. It is not renderer specific at all. It's just an Arena that >> recycles freed objects. Maybe we should move that functionality into the new >> Arena class. But RenderArena is used all over the place, so maybe that's >> going one step too far down this road? _______________________________________________ webkit-dev mailing list webkit-dev@lists.webkit.org http://lists.webkit.org/mailman/listinfo.cgi/webkit-dev
