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 leaks were detected also in the demo project #3671

Closed
1 task done
JunsW opened this issue Sep 30, 2018 · 3 comments
Closed
1 task done

Memory leaks were detected also in the demo project #3671

JunsW opened this issue Sep 30, 2018 · 3 comments
Assignees

Comments

@JunsW
Copy link

JunsW commented Sep 30, 2018

What did you do?

Displaying pie chart, line chart, horizontal bar chart in my project.

What did you expect to happen?

No memory leak.

What happened instead?

Massive memory leaks were found when I used instruments to check my project that contains lots of different charts.
The similar leaks were found when I used instruments to check the demo project.

Those leaks were found around the initialization of ChartViewBase, DefaultValueFormatter and ChartDataEntry.
The rest seems to be related to the data set.

Demo Leaks:
demo_leaks_backtree
demo_leaks_calltree
My Project Leaks:
myproject_leaks_calltree

Charts Environment

Charts version/Branch/Commit Number: 3.2.0 (Mine ) / 3.1.1 (Demo)
Xcode version: Version 10.0 (10A254a)
Swift version: 4.2 (Mine) / 4.0 (Demo)
Platform(s) running Charts: iOS
macOS version running Xcode: 10.13.6

Demo Project

Leaks should be caught after randomly clicked several charts in the demo list.

@StefaniOSApps
Copy link

me@same issue ...

@RamblinWreck77
Copy link

I had to roll back to 3.1.1 after testing out 3.2.0 because I found the same problem.

@liuxuan30
Copy link
Member

please check if #3680 fixed it

@liuxuan30 liuxuan30 self-assigned this Oct 13, 2018
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

4 participants