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

* [android] fix : list OOM cause by viewHolder cache,set the upper limit for the viewHolder cache pool #512

Merged
merged 2 commits into from Jul 21, 2017

Conversation

Projects
None yet
4 participants
@miomin
Contributor

miomin commented Jul 17, 2017

bugfix : list OOM cause by viewHolder cache,set the upper limit for the viewHolder cache pool

@weex-bot

This comment has been minimized.

Show comment
Hide comment
@weex-bot

weex-bot Jul 17, 2017

Fails
🚫 This PR modify SDK code. Please add/modify corresponding testcases. If it is ok, please comment about it. Or put '@notdanger' in you commit message.
Warnings
⚠️ No Changelog changes!
⚠️ This PR should update related documents as well.
Messages
📖 According to the blame info, we recommended @sospartan , @cxfeng1 to be the reviewers.

Generated by 🚫 dangerJS

weex-bot commented Jul 17, 2017

Fails
🚫 This PR modify SDK code. Please add/modify corresponding testcases. If it is ok, please comment about it. Or put '@notdanger' in you commit message.
Warnings
⚠️ No Changelog changes!
⚠️ This PR should update related documents as well.
Messages
📖 According to the blame info, we recommended @sospartan , @cxfeng1 to be the reviewers.

Generated by 🚫 dangerJS

@zshshr

zshshr approved these changes Jul 17, 2017

@miomin miomin changed the title from * [android] fix : set the upper limit for the viewHolder cache pool to * [android] fix : list OOM cause by viewHolder cache,set the upper limit for the viewHolder cache pool Jul 17, 2017

@zshshr

zshshr approved these changes Jul 21, 2017

@asfgit asfgit merged commit 4ba3101 into apache:0.15-dev Jul 21, 2017

1 check failed

continuous-integration/travis-ci/pr The Travis CI build failed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment