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

Legend leak in version 3.0.1 #2425

Closed
laizf opened this issue May 10, 2017 · 11 comments
Closed

Legend leak in version 3.0.1 #2425

laizf opened this issue May 10, 2017 · 11 comments

Comments

@laizf
Copy link

laizf commented May 10, 2017

image
When use instruments to analyze memory leak, found Legend is leaked, but cannot find the reason. The stack trace is shown in the image above.

@laizf
Copy link
Author

laizf commented May 10, 2017

It's seems the same bug with issue:#2416 "Memory Leak in BarChartView Data".
image

@laizf
Copy link
Author

laizf commented May 10, 2017

My XCode Version is 8.3 (8E162). I ever used old version XCode to check memory leak about two month ago, and no leak found that time. I think you should update your XCode to the latest version if you want to reproduce this bug.

@thierryH91200
Copy link
Contributor

this is the latest version sorry

capture d ecran 2017-05-10 a 10 38 12

@laizf
Copy link
Author

laizf commented May 10, 2017

And can you reproduce or solve this problem? @thierryH91200

@laizf
Copy link
Author

laizf commented May 10, 2017

@liuxuan30

@liuxuan30
Copy link
Member

liuxuan30 commented May 15, 2017

Update:
See my comments below, when pop back, the same leaked object is freed

OK, legend seems do have a leak, but I can't reproduce #2416, so let's make it separate.
image

@laizf
Copy link
Author

laizf commented May 15, 2017

The ref count is -1 finally, but instruments still mark it as leak. The instruments file is too large to upload, so I just show you the screenshot:
image
image

@liuxuan30
Copy link
Member

liuxuan30 commented May 15, 2017

No, I tried when you pop back, the leaked object is freed then:
image

When your chart is displaying, I can't say it's our problem, as when the chart get destroyed, the objects saying leak is also freed.

You can try on your side, just pop back or make your chart dealloc

Last year at WWDC 2016, apple's people also confirm some of the false leak is their bug (radar 26817765), just like this. So please double check.

I have filed a radar 32189235 to Apple

@laizf
Copy link
Author

laizf commented May 15, 2017

Copy

@liuxuan30
Copy link
Member

If you sure it's freed, I will close the related issues.

@laizf
Copy link
Author

laizf commented May 15, 2017

I think so!
image

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

No branches or pull requests

3 participants