[Webkit-unassigned] [Bug 29256] Use MemoryManager to gather platform-specific low level memory requests

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Tue Sep 15 12:41:43 PDT 2009


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


Geoffrey Garen <ggaren at apple.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |ggaren at apple.com




--- Comment #4 from Geoffrey Garen <ggaren at apple.com>  2009-09-15 12:41:42 PDT ---
What's the goal of this patch? JavaScriptCore already supports malloc/free and
Windows.

I disagree with the approach here.

1. There's a layering problem. An "abstract" memory allocator should be
agnostic about the kinds of allocations its clients will make. But this
allocator has functions like allocateJSBlock and reserveJSRegisterFile, tying
it to its clients. This seems like a confused and unscalable design.

2. You haven't ported any existing platforms to the abstract interface, so
you've just added yet another way to allocate memory, complicating the design.

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



More information about the webkit-unassigned mailing list