Skip to content
This repository has been archived by the owner on Feb 1, 2025. It is now read-only.

[Fix] conditionally set Symbol.toStringTag, since it's nonwritable #400

Merged
merged 3 commits into from
Jul 21, 2020
Merged

[Fix] conditionally set Symbol.toStringTag, since it's nonwritable #400

merged 3 commits into from
Jul 21, 2020

Conversation

ljharb
Copy link
Contributor

@ljharb ljharb commented Jul 21, 2020

This avoids an exception when the new iterator helpers proposal provides a spec-compliant nonwritable Symbol.toStringTag property.

See tc39/proposal-iterator-helpers#115 and https://bugzilla.mozilla.org/show_bug.cgi?id=1644581#c6.

Closes #399.

This avoids an exception when the new iterator helpers proposal provides a spec-compliant nonwritable Symbol.toStringTag property.
Copy link
Contributor

@benjamn benjamn left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me. Thanks to @ljharb for bringing this to my attention, and to @ExE-Boss for the initial PR with the test (which I cherry-picked into this PR).

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants