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

Fix Memory Leaks #368

Merged
merged 7 commits into from
May 12, 2017
Merged

Fix Memory Leaks #368

merged 7 commits into from
May 12, 2017

Conversation

sarahsnow1
Copy link
Member

Overview

Fixes memory leaks!

Proposed Changes

  • Upgrades to LOST 2.3.0-SNAPSHOT which always uses the application Context when creating new LostApiClients
  • LocationFactory is simplified and improved. It does not keep a static reference to a Context and instead vends a new client only if one doesn't already exist
  • Static inner class in OverlayManager now uses a weak reference of OverlayManager
  • Migrates MapzenManager to use the application context
  • Updates tests for new LOST version and for Context updates
  • Small cleanup in basic SDK sample Activity

Closes #338

@sarahsnow1 sarahsnow1 merged commit ad7898b into master May 12, 2017
@sarahsnow1 sarahsnow1 deleted the 338-memory-leak branch May 12, 2017 17:29
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.

3 participants