Skip to content
This repository has been archived by the owner on Aug 8, 2023. It is now read-only.

onMapChanged churns memory #2678

Closed
ljbade opened this issue Oct 20, 2015 · 1 comment
Closed

onMapChanged churns memory #2678

ljbade opened this issue Oct 20, 2015 · 1 comment
Assignees
@ljbade
Labels
Android Mapbox Maps SDK for Android
Milestone
android-v2.2.0

Comments

@ljbade
Copy link
Contributor

ljbade commented Oct 20, 2015

Noticed that onMapChange creates a lot of new Runnable object.

If we ensure we only call renderSync from the UI thread, we can ditch the post().
@ljbade ljbade added the Android Mapbox Maps SDK for Android label Oct 20, 2015
@ljbade ljbade added this to the android-v2.2.0 milestone Oct 20, 2015
@ljbade
Copy link
Contributor Author

ljbade commented Oct 20, 2015

It seems the iterator() for the list of event listeners also churns memory when using gestures.

ljbade pushed a commit that referenced this issue Oct 20, 2015
[android] Sync GPS marker
97d193b 
Lots of other performance and memory improvements.
Fixes #2668
Fixes #2411
Fixes #2678
Fixes #2675
@ljbade ljbade mentioned this issue Oct 20, 2015
Closed
@ljbade ljbade self-assigned this Oct 20, 2015
ljbade pushed a commit that referenced this issue Oct 20, 2015
[android] Sync GPS marker
bc85f93 
Lots of other performance and memory improvements.
Fixes #2668
Fixes #2411
Fixes #2678
Fixes #2675
@ljbade ljbade mentioned this issue Oct 22, 2015
Merged
ljbade pushed a commit that referenced this issue Oct 22, 2015
[android] Implement UserLocationView
8b79a8d 
Draws animated and synced GPS marker.
Inlcudes direction arrow and accuracy ring.
Fade out the compass after a delay when reset to north.
Lots of other performance and memory improvements.
add OnMyLocationChangeListener
Move FAB with snackbar
Fix getMetersPerPixelAtLatitude
Fix some incorrect annotations
Fixes #2668
Fixes #2411
Fixes #2678
Fixes #2675
Fixes #2566
Fixes #2549
Fixes #2692
Fixes #2572
AndwareSsj pushed a commit to AndwareSsj/mapbox-gl-native that referenced this issue Nov 6, 2015
[android] Implement UserLocationView
f64079a 
Draws animated and synced GPS marker.
Inlcudes direction arrow and accuracy ring.
Fade out the compass after a delay when reset to north.
Lots of other performance and memory improvements.
add OnMyLocationChangeListener
Move FAB with snackbar
Fix getMetersPerPixelAtLatitude
Fix some incorrect annotations
Fixes mapbox#2668
Fixes mapbox#2411
Fixes mapbox#2678
Fixes mapbox#2675
Fixes mapbox#2566
Fixes mapbox#2549
Fixes mapbox#2692
Fixes mapbox#2572
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@ljbade ljbade

Labels
Android Mapbox Maps SDK for Android
Projects
None yet
Milestone
android-v2.2.0
Development

No branches or pull requests
1 participant
@ljbade