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 doesn't release when after map render #226

Closed
thuydao opened this issue Feb 17, 2020 · 8 comments
Closed

Memory doesn't release when after map render #226

thuydao opened this issue Feb 17, 2020 · 8 comments

Comments

@thuydao
Copy link

thuydao commented Feb 17, 2020

For example, I go to user interface > back to main screen > go to user interface again. The memory not reduce.

@lj64
Copy link

lj64 commented Jul 1, 2020

I have the same problem.

App is crashing on ios when using the map a lot.
It is very easy to reproduce in example app.
Just enter one page with map, go back, enter again, go back.
Repeat several times and app will crash.

In my case on iphone 6s:

  • open and close Full screen map 47 times, then crash
  • open and close User interface 28 times, then crash

In my project it takes 16-18 times to open and close map page before it crashes.
The only log from xcode that I had after app crashing was:
Message from debugger: Terminated due to memory issue

@kleeb
Copy link
Contributor

kleeb commented Jul 2, 2020

+1

@m0nac0
Copy link
Collaborator

m0nac0 commented Jul 2, 2020

@thuydao @kleeb Is this only on iOS?

@kleeb
Copy link
Contributor

kleeb commented Jul 2, 2020

Yes. I reproduced that even on the example app

@kuhnroyal
Copy link

This seems to happen only on iOS. Does anyone know if this happens in the flutter-mapbox-gl plugin or in the native code?

@kuhnroyal
Copy link

Inspecting this with XCode instruments you can see the MapboxMapController does not get cleaned up. The reference count keeps going up.

@thirteenthstep
Copy link
Contributor

Pull Request #370 should fix this

@tobrun
Copy link
Collaborator

tobrun commented Nov 7, 2020

#370 merged, please reach out if still an issue

@tobrun tobrun closed this as completed Nov 7, 2020
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

7 participants