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

port backendscope changes to release branch #9538

Merged

Conversation

ivovandongen
Copy link
Contributor

Ports backend scope changes that make manual reference counting on android for backend activation/deactivation obsolete.

Fixes #9458

@zugaldia I tagged you for review as you reported #9458 and I'm not sure how to reproduce.

- Guards against duplicate activations by checking wether the backend of the prior scope is the same as the current ones
- Makes sure that only the most outer backend scope deactivates by tracking activation state
@ivovandongen ivovandongen added Android Mapbox Maps SDK for Android Core The cross-platform C++ core, aka mbgl labels Jul 18, 2017
@ivovandongen ivovandongen self-assigned this Jul 18, 2017
@ivovandongen ivovandongen changed the title port backendscope changes port backendscope changes to release branch Jul 18, 2017
@ivovandongen ivovandongen requested a review from 1ec5 July 18, 2017 15:11
@ivovandongen
Copy link
Contributor Author

@1ec5 I've tested this on iOS. Tests pass and the app works as expected still. Tagged you for review to be sure.

@ivovandongen ivovandongen merged commit e7f45e0 into release-ios-v3.6.0-android-v5.1.0 Jul 19, 2017
@ivovandongen ivovandongen deleted the 9458-port-backendscope-changes branch July 19, 2017 15:34
@tobrun tobrun added this to the android-v5.1.1 milestone Jul 19, 2017
@tobrun tobrun mentioned this pull request Jul 20, 2017
15 tasks
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Android Mapbox Maps SDK for Android Core The cross-platform C++ core, aka mbgl
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants