Bug 204587
Summary: | [bmalloc] Same variable name `sizeClassCount` is used differently in Heap.cpp | ||
---|---|---|---|
Product: | WebKit | Reporter: | Basuke Suzuki <basuke> |
Component: | bmalloc | Assignee: | Nobody <webkit-unassigned> |
Status: | RESOLVED INVALID | ||
Severity: | Normal | CC: | ggaren |
Priority: | P2 | ||
Version: | WebKit Nightly Build | ||
Hardware: | Unspecified | ||
OS: | Unspecified |
Basuke Suzuki
It is confusing.
Attachments | ||
---|---|---|
Add attachment proposed patch, testcase, etc. |
Basuke Suzuki
In Heap::initializeLineMetadata(), sizeClassCount is defined and initialized as bmalloc::sizeClass(smallLineSize). This variable is already defined in Sizes.h and used in different meaning.
It is totally meaningful to rename variable name in initializeLineMetadata() from sizeClassCount to just classCount.
Basuke Suzuki
This name was changed while this patch landing. https://bugs.webkit.org/show_bug.cgi?id=206044