Skip to content

Commit

Permalink
Bug 1738020 part 2 - Remove stale comment, fix assertion. r=tcampbell
Browse files Browse the repository at this point in the history
This likely hasn't been true for quite some time.

Depends on D129628

Differential Revision: https://phabricator.services.mozilla.com/D129629
  • Loading branch information
jandem committed Oct 29, 2021
1 parent 5fe28a4 commit 0caab75
Showing 1 changed file with 2 additions and 3 deletions.
5 changes: 2 additions & 3 deletions js/src/vm/EnvironmentObject.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -1015,9 +1015,8 @@ BlockLexicalEnvironmentObject* BlockLexicalEnvironmentObject::clone(
return nullptr;
}

// We can't assert that the clone has the same shape, because it could
// have been reshaped by ReshapeForShadowedProp.
MOZ_ASSERT(env->slotSpan() == copy->slotSpan());
MOZ_ASSERT(env->shape() == copy->shape());

for (uint32_t i = JSSLOT_FREE(&class_); i < copy->slotSpan(); i++) {
copy->setSlot(i, env->getSlot(i));
}
Expand Down

0 comments on commit 0caab75

Please sign in to comment.