Skip to content

Commit fd80dfe

Browse files
ast-huggerast-hugger
committed
Fix the doc comment of BASE_CONTIGUOUS_VECTOR_LEN
https://bugs.webkit.org/show_bug.cgi?id=314158 rdar://176323440 Reviewed by Yusuke Suzuki. The comment says "BASE_XXX_VECTOR_LEN is the maximum number of vector elements ..." but it's actually the minimum number of elements. Testing: not applicable - doc-only Canonical link: https://commits.webkit.org/312705@main
1 parent 4d3dfed commit fd80dfe

1 file changed

Lines changed: 2 additions & 2 deletions

File tree

‎Source/JavaScriptCore/runtime/ArrayConventions.h‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -75,8 +75,8 @@ namespace JSC {
7575
static_assert(MIN_SPARSE_ARRAY_INDEX <= MAX_STORAGE_VECTOR_INDEX, "MIN_SPARSE_ARRAY_INDEX must be less than or equal to MAX_STORAGE_VECTOR_INDEX");
7676
static_assert(MAX_STORAGE_VECTOR_INDEX <= MAX_ARRAY_INDEX, "MAX_STORAGE_VECTOR_INDEX must be less than or equal to MAX_ARRAY_INDEX");
7777

78-
// The value BASE_XXX_VECTOR_LEN is the maximum number of vector elements we'll allocate
79-
// for an array that was created with a sepcified length (e.g. a = new Array(123))
78+
// The value BASE_XXX_VECTOR_LEN is the minimum number of vector elements we'll allocate
79+
// for an array that was created with a specified length (e.g. a = new Array(123))
8080
#define BASE_CONTIGUOUS_VECTOR_LEN 3U
8181
#define BASE_CONTIGUOUS_VECTOR_LEN_EMPTY 5U
8282
#define BASE_CONTIGUOUS_VECTOR_LEN_MIN 3U

0 commit comments

Comments
 (0)