New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Mark WeakMap
s of private fields as pure
#13194
Mark WeakMap
s of private fields as pure
#13194
Conversation
Build successful! You can test your changes in the REPL here: https://babeljs.io/repl/build/45529/ |
We should also annotate https://github.com/babel/babel/pull/13172/files#diff-8a6df468afdcc49425d50abb711a0b1439b2120a6c51814cc563ca10ba7730f1R1 as pure. |
This pull request is automatically built and testable in CodeSandbox. To see build info of the built libraries, click here or the icon next to each commit SHA. Latest deployment of this branch, based on commit 5c6fb6b:
|
I'll rebase this on top of the Babel 7.14 branch and do it in this PR |
Is this commit from yesterday 66181db intentionally in the MR? |
Nope, I need to rebase |
5c6fb6b
to
6b14bdc
Compare
The
/*#__PURE__*/
annotation lets terser strip away thoseWeakMap
s when the class is unused. TheclassPrivateFieldLooseKey
isn't technically pure (since it uses a global counter), but it can still be safely removed by terser.