Bug 77116

Summary: Merge AllocationSpace into MarkedSpace
Product: WebKit Reporter: Mark Hahnenberg <mhahnenberg>
Component: JavaScriptCoreAssignee: Mark Hahnenberg <mhahnenberg>
Status: RESOLVED FIXED    
Severity: Normal CC: levin+threading, rakuco, webkit.review.bot
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch
none
Fixing builds
none
Fixing more build stuff
none
Hopefully last build fix none

Mark Hahnenberg
Reported 2012-01-26 10:27:17 PST
We have two separate classes that handle allocation in MarkedBlocks. It's unclear why we make the distinction between AllocationSpace and MarkedSpace. In preparation for future refactoring/enhancement of MarkedSpace allocation, we should merge AllocationSpace into MarkedSpace.
Attachments
Patch (30.29 KB, patch)
2012-01-26 10:37 PST, Mark Hahnenberg 		no flags
DetailsFormatted DiffDiff
Fixing builds (33.02 KB, patch)
2012-01-26 10:54 PST, Mark Hahnenberg 		no flags
DetailsFormatted DiffDiff
Fixing more build stuff (35.02 KB, patch)
2012-01-26 15:47 PST, Mark Hahnenberg 		no flags
DetailsFormatted DiffDiff
Hopefully last build fix (35.08 KB, patch)
2012-01-26 16:08 PST, Mark Hahnenberg 		no flags
DetailsFormatted DiffDiff
Show Obsolete (3) View All Add attachment proposed patch, testcase, etc.
Mark Hahnenberg
Comment 1 2012-01-26 10:37:43 PST
Created attachment 124141 [details] Patch
Mark Hahnenberg
Comment 2 2012-01-26 10:54:53 PST
Created attachment 124145 [details] Fixing builds
Geoffrey Garen
Comment 3 2012-01-26 10:57:33 PST
Comment on attachment 124145 [details] Fixing builds rs=me
Early Warning System Bot
Comment 4 2012-01-26 11:17:25 PST
Comment on attachment 124145 [details] Fixing builds Attachment 124145 [details] did not pass qt-ews (qt): Output: http://queues.webkit.org/results/11357002
Gyuyoung Kim
Comment 5 2012-01-26 11:25:25 PST
Comment on attachment 124145 [details] Fixing builds Attachment 124145 [details] did not pass efl-ews (efl): Output: http://queues.webkit.org/results/11191414
Mark Hahnenberg
Comment 6 2012-01-26 15:47:17 PST
Created attachment 124198 [details] Fixing more build stuff
Mark Hahnenberg
Comment 7 2012-01-26 16:08:54 PST
Created attachment 124204 [details] Hopefully last build fix
Geoffrey Garen
Comment 8 2012-01-26 16:09:44 PST
Comment on attachment 124204 [details] Hopefully last build fix rs=me
WebKit Review Bot
Comment 9 2012-01-26 18:30:54 PST
Comment on attachment 124204 [details] Hopefully last build fix Clearing flags on attachment: 124204 Committed r106078: <http://trac.webkit.org/changeset/106078>
WebKit Review Bot
Comment 10 2012-01-26 18:30:59 PST
All reviewed patches have been landed. Closing bug.
Note You need to log in before you can comment on or make changes to this bug.