From 859e3145703c6627f9a353162e3a216fae782634 Mon Sep 17 00:00:00 2001 From: Lei Shi Date: Tue, 16 Jan 2018 16:18:02 -0800 Subject: [PATCH] deps: update ChakraCore to Microsoft/ChakraCore@79ef8eda34 [1.8>1.9] [MERGE #4528 @leirocks] Fix bad assertion in DirectSetItemAtRange for OP_MEMSET Merge pull request #4528 from leirocks:op_memset Reviewed-By: chakrabot --- .../core/lib/Runtime/Library/JavascriptArray.inl | 12 ++---------- 1 file changed, 2 insertions(+), 10 deletions(-) diff --git a/deps/chakrashim/core/lib/Runtime/Library/JavascriptArray.inl b/deps/chakrashim/core/lib/Runtime/Library/JavascriptArray.inl index 0311267d35b..f3e393bcbe4 100644 --- a/deps/chakrashim/core/lib/Runtime/Library/JavascriptArray.inl +++ b/deps/chakrashim/core/lib/Runtime/Library/JavascriptArray.inl @@ -1160,23 +1160,15 @@ SECOND_PASS: } else if (startIndex == 0 && length > this->length && (head == EmptySegment || length > head->size)) { - Recycler *recycler = GetRecycler(); - this->length = length; - this->EnsureHead(); - SparseArraySegmentBase* current = nullptr; - - Assert(head->size < length); - - current = SparseArraySegment::AllocateSegment(recycler, 0, length, (SparseArraySegment *)nullptr); + SparseArraySegmentBase* current = SparseArraySegment::AllocateSegment(recycler, 0, length, (SparseArraySegment *)nullptr); this->SetHeadAndLastUsedSegment(current); + this->length = length; Assert(!HasSegmentMap()); SetHasNoMissingValues(true); - CopyValueToSegmentBuferNoCheck(((Js::SparseArraySegment*)current)->elements, length, newValue); - this->SetLastUsedSegment(current); } else {