Skip to content
This repository has been archived by the owner. It is now read-only.

[WEEX-547][jsfm] Remove module proxy cache of weex.requireModule #1389

Merged
merged 1 commit into from Aug 2, 2018

Conversation

@Hanks10100
Copy link
Contributor

@Hanks10100 Hanks10100 commented Aug 2, 2018

JS Framework has a cache strategy for module proxies which is good for performance. But it also a potential reason for the memory leak.

@weex-bot
Copy link

@weex-bot weex-bot commented Aug 2, 2018

Messages
📖 No android file has been changed.
📖 android build verification finished.

Generated by 🚫 dangerJS

imyzf
imyzf approved these changes Aug 2, 2018
@Hanks10100 Hanks10100 merged commit 053f26e into apache:master Aug 2, 2018
1 check passed
yxping added a commit to yxping/incubator-weex that referenced this issue Aug 9, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

3 participants