[Webkit-unassigned] [Bug 65840] Fix comment in NewSpace.h

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Tue Aug 9 23:29:54 PDT 2011


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





--- Comment #6 from Zoltan Herczeg <zherczeg at webkit.org>  2011-08-09 23:29:54 PST ---
> Where to allocate is one question. Independently, this comment describes the size of the cells in the class. That size is 128, not 128 - preciseStep.

Are you sure? The comment said:

// [ 128, 256... 1024 )

The size of the first block is really 128, but the second is 128 as well, not 256, and the last is 128 as well, not 1024. Instead I think the comment describes the ranges, the first range is [128-256), the second is [256-384) ...

The growth rate is clearly linear:

static const size_t preciseCutoff = 128;
static const size_t impreciseStep = preciseCutoff;
return m_impreciseSizeClasses[(bytes - 1) / impreciseStep];

-- 
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