Skip to content
This repository has been archived by the owner on Apr 9, 2022. It is now read-only.

replace @class to @__PURE__ annotation. #186

Closed
aitboudad opened this issue Sep 30, 2017 · 1 comment
Closed

replace @class to @__PURE__ annotation. #186

aitboudad opened this issue Sep 30, 2017 · 1 comment

Comments

@aitboudad
Copy link

see mishoo/UglifyJS#2279

@clydin
Copy link
Member

clydin commented Sep 30, 2017

That’s unfortunately not a safe operation. All classes are not guaranteed to be side effect free. The build optimizer currently analyzes the code structure to determine the presence of a downlevel class and further work is being considered to determine the presence of potential side effects to improve the safety of the optimization.

It’s also important to note that the use of pure is a misnomer in this situation and an unnecessary threshold for the intended optimization.

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

No branches or pull requests

2 participants