Skip to content
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

Memory Leak #11

Merged
merged 1 commit into from
May 10, 2016
Merged

Memory Leak #11

merged 1 commit into from
May 10, 2016

Conversation

julioyg
Copy link

@julioyg julioyg commented May 9, 2016

Issue: #10

Using you library in our project (thanks a lot for it) in which we use CanaryLeak I've found two leaks in your code:

  1. There are some anonymous AnimatorUpdateListener defined that holds a reference to the context.
  2. There are some anonymous TypeEvaluator implementations that because they holds references to their parent class and this one to the context leak memory too.

@julioyg julioyg mentioned this pull request May 9, 2016
@tajchert tajchert merged commit e20b6d1 into tajchert:master May 10, 2016
@julioyg
Copy link
Author

julioyg commented May 19, 2016

sorry @tajchert is there a new version available in jcenter or any planning on it?

@fifanrg
Copy link

fifanrg commented Jul 8, 2016

Thanks for your awesome lib !!
Same question here , is there a new version available in jcenter for this memory fix ?

Thanks.

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

Successfully merging this pull request may close these issues.

4 participants